Skip to content

fix(factory): replace context spills with manifests - #198

Merged
walker-tx merged 1 commit into
mainfrom
fix/manifest-guide-context
Aug 29, 2026
Merged

fix(factory): replace context spills with manifests#198
walker-tx merged 1 commit into
mainfrom
fix/manifest-guide-context

Conversation

@walker-tx

Copy link
Copy Markdown
Member

Eliminates model-managed Phase 1 spill consumption.

The context inspector now returns a ~1.5 KB validated manifest of approved paths and character counts instead of ~170 KB of raw contents requiring roughly 58 stateful reads. Downstream agents receive explicit phase-specific manifest paths and read only those files directly from /workspace.

Security properties retained and strengthened:

  • canonical path and symlink rejection
  • fixed allowlist and mandatory authority/persona/role/schema entries
  • 40-file and 7,000-byte output bounds
  • no raw context content in the manifest
  • no repository discovery by child agents

Validation:

  • full factory suite 13/13
  • ShellCheck
  • actionlint
  • independent review: PASS

@walker-tx
walker-tx merged commit c74cd7a into main Aug 29, 2026
1 check passed
@walker-tx
walker-tx deleted the fix/manifest-guide-context branch August 29, 2026 15:51
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.

1 participant