Conversation
There was a problem hiding this comment.
Pull request overview
This PR resets the Contextrie repository to a clean monorepo structure with a minimal core package that defines type contracts and source primitives. The changes remove all implementation code (client, CLI, demo) and replace it with a foundational type system that separates concerns between core contracts and future adapters.
Changes:
- Removed all implementation code (ingester, assessor, composer, client, CLI, demo) and dependencies
- Added core type system with source primitives (Document, List, ReferenceDocument, ReferenceList) and ingestion contracts
- Set up SvelteKit documentation site scaffold and stub directories for CLI, examples, and Python package
Reviewed changes
Copilot reviewed 75 out of 82 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Removed from root (no longer a monolithic package) |
| tsconfig.build.json | Removed from root |
| index.ts | Simplified to only export core types |
| readme.md | Updated with new repo layout and simplified API concepts |
| core/types/ | New core type system defining source primitives and ingestion contracts |
| core/package.json | New standalone package configuration for core |
| docs/ | New SvelteKit documentation site scaffold |
| cli/README.md | Stub placeholder |
| examples/README.md | Stub placeholder |
| python/README.md | Stub placeholder |
| demo/ | Completely removed with all demo files |
| client/ | Removed all client implementation |
| core/ingester/, core/assessor/, core/compose/, core/session/ | All removed |
| .github/workflows/pr-checks.yml | Removed CI workflow |
| contributing.md | Removed |
| AGENTS.md | Removed |
| eslint.config.js | Removed from root |
| .gitignore | Simplified |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@feuersteiner I've opened a new pull request, #41, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@feuersteiner I've opened a new pull request, #42, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: feuersteiner <18667704+feuersteiner@users.noreply.github.com>
Co-authored-by: feuersteiner <18667704+feuersteiner@users.noreply.github.com>
Remove non-existent `the-plan.md` reference from repo layout
Make `kind` property readonly in `ReferenceListSource`
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 75 out of 82 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 75 out of 82 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const gitignorePath = path.resolve(import.meta.dirname, '.gitignore'); | ||
|
|
There was a problem hiding this comment.
import.meta.dirname is not a standard ESM property in Node (and differs from Bun’s import.meta.dir). As a result, loading this ESLint config will likely throw at runtime, breaking eslint .. Use a portable dirname derivation (e.g., from import.meta.url) or switch to the runtime-supported import.meta.dir if this is Bun-only.
b7db489 to
5994ee8
Compare
Concise rebuild plan for Contextrie.
Goals
Tasks