Skip to content

feat(catalyst): preserve complete Phase 1 context evidence - #77

Merged
pmanko merged 3 commits into
mainfrom
codex/phase1-context-evidence
Aug 26, 2026
Merged

feat(catalyst): preserve complete Phase 1 context evidence#77
pmanko merged 3 commits into
mainfrom
codex/phase1-context-evidence

Conversation

@pmanko

@pmanko pmanko commented Aug 26, 2026

Copy link
Copy Markdown
Member

Summary

  • supplies every prior user instruction, every eligible verified example, the relevant prior failure, and any experimental guidance without item-count caps, ranking, or silent trimming
  • records the exact Hub request, token measurement, request digest, and context-capacity rejection for each writer and reviewer call
  • keeps validator findings advisory, requires successful execution before a kept query becomes an example, and never sends result rows
  • defines retained history as user instructions plus structured query, example, and failure records; raw model replies are not replayed as trusted context
  • removes the remaining silent failure-detail truncation and stale guidance-precedence wording

Dependencies

This is stacked on #76 for the readable PostgreSQL surface. Exact request evidence is supplied by pmanko/med-agent-hub#23.

Validation

  • full Catalyst Gateway suite: 348 passed, 3 skipped
  • focused context/request-evidence suite: 132 passed
  • Ruff format and lint: clean
  • JSON schemas and git diff: clean

Boundaries

There is no fixed relation count or metadata whitelist. If the complete physical request exceeds a selected model's measured context capacity, the request is rejected and recorded rather than silently reduced and retried. No database, seed, deployment, or live-environment changes are included.

@pmanko

pmanko commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

@copilot review

Please review this stacked R6 change against the Phase 1 decisions in the PR body. In particular, check that it preserves complete eligible context and exact per-model request evidence without introducing relation-count limits, context-item caps, ranking, silent truncation, or a required Pin interface. Please do not treat historical wire names such as approvedViews or pinned/unpinned as current product policy.

Copilot AI commented Aug 26, 2026

Copy link
Copy Markdown

@copilot review

Please review this stacked R6 change against the Phase 1 decisions in the PR body. In particular, check that it preserves complete eligible context and exact per-model request evidenc...

Reviewed and updated in 75ccb1d. The implementation already preserves complete eligible context and exact per-invocation Hub request evidence without ranking, caps, or silent trimming; I only corrected the lingering test wording that implied a required pin-style product interface.

@pmanko
pmanko force-pushed the codex/phase1-context-evidence branch from 75ccb1d to 836b179 Compare August 26, 2026 04:56
@pmanko
pmanko changed the base branch from codex/phase1-readable-surface to main August 26, 2026 06:45
@pmanko
pmanko force-pushed the codex/phase1-context-evidence branch from 836b179 to e880e5a Compare August 26, 2026 06:46
@pmanko
pmanko merged commit 9b1429f into main Aug 26, 2026
5 checks passed
@pmanko
pmanko deleted the codex/phase1-context-evidence branch August 26, 2026 07:33
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