Skip to content

feat: local knowledge library, source discovery and session organization - #45

Merged
guicybercode merged 32 commits into
mainfrom
feat/xirp-local-workflows
Sep 6, 2026
Merged

feat: local knowledge library, source discovery and session organization#45
guicybercode merged 32 commits into
mainfrom
feat/xirp-local-workflows

Conversation

@guicybercode

@guicybercode guicybercode commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Adds a local prompt/context library, rule and skill inspection, and persistent project/session organization. Global or project text can be searched, edited with revision checks and inserted into the canvas terminal composer as an explicit editable snapshot. Pin/archive/workflow metadata is separate from process state: archiving or marking a workflow done leaves a live session running.

  • Knowledge uses the existing SQLite/daemon/IPC path with migration 0004, bounded pages and optimistic revisions. The canvas library/composer mounting is reconciled from the base branch; insertion never implicitly sends or starts a session.
  • Rule/skill discovery exposes bounded metadata and explicit read-only previews through expiring opaque capabilities. Nested project directories retain relative owner scope and explicit pruning, with shared limits and root/global priority. Known skill-directory links are captured safely; file/parent replacement invalidates reads. Native loading and activation are not inferred.
  • Organization uses migration 0005, bounded ordered batches, revision-zero defaults and whole-record compare-and-swap. Isolated controls preserve drafts through conflicts; canvas mounting, pin sorting and archive filtering remain coordinated with S1.
  • All wire names and TypeScript/JSON mirrors are reconciled with worktree.list. Initial-objective execution and trustworthy native resume/fork remain S2 runtime integration work. Portal-dependent features are recorded as dependencies.

Validation on macOS: 300 frontend tests, TypeScript, production build and 13 Chromium E2E passed. Core/daemon/agents suites passed; storage 45 tests passed separately. Five-package all-target Clippy and formatting passed. The latest inherited probe correction passed all 64 agents tests and Clippy. SQLite/socket tests cover persistence, concurrent revisions, bounded reads, replacement rejection and organization changes while a real SessionManager-owned child stays running.

The expanded PTY suite exposed one pre-existing cleanup failure: the manager-drop test left a live shell, although its isolated rerun passed. Read-only investigation reproduced an out-of-order process-cache defect. A review patch with 11 passing standalone module tests is in docs/codex/proposals/pty-process-cache-ordering.patch for S2; it has not changed the runtime, and no test assertion was weakened. Full backend/native acceptance is therefore still open. Linux/macOS CI and Packaging both passed at 41f679c: https://github.com/guicybercode/Jig/actions/runs/34005300527 and https://github.com/guicybercode/Jig/actions/runs/34005300530. Nested discovery adds 9 scanner and 1 real-socket regression; all 31 knowledge daemon tests and daemon Clippy pass. The mounted source inspector is reconciled from S1; new head CI follows push. Evidence and exact integration APIs: docs/codex/xirp-context-report.md and docs/xirp-local-parity.md.

Keep the assertion tied to the two-second supervisor interval while using the production-like one-second reap deadline from the shared test configuration. This avoids treating a loaded macOS runner's inability to reap within 50 ms as a runtime failure.
… feat/xirp-local-workflows

# Conflicts:
#	docs/codex/xirp-integration-request.md
@guicybercode guicybercode changed the title feat: add local prompts and reusable context library feat: local knowledge library, source discovery and session organization Sep 6, 2026
@guicybercode
guicybercode changed the base branch from refactor/canvas-only-shell to main September 6, 2026 02:22
Combine the scoped local file service with the knowledge discovery and canvas changes already merged into main, plus the monotonic process snapshot correction from PR42. Preserve all IPC catalog mirrors, file safety behavior, and regression coverage.\n\nValidated TypeScript, 319 frontend tests, 13 file contract tests, 12 file IPC tests, 21 knowledge tests, Rust formatting and diff checks.
Reconcile the local knowledge and organization workflows with the prepared PR44 file service and PR42 process snapshot correction. Preserve both IPC capabilities, the organization migration, all existing UI behavior and regression tests.

Validated TypeScript, 336 frontend tests, version consistency, Rust formatting and diff checks. Full CI and Packaging validate the combined head on Linux and macOS.
@guicybercode
guicybercode marked this pull request as ready for review September 6, 2026 02:32
Copilot AI lite review requested due to automatic review settings September 6, 2026 02:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@guicybercode
guicybercode merged commit c1be3da into main Sep 6, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants