Skip to content

fix(coding-agents): route repository surveys explicitly - #3343

Open
TheAngryPit wants to merge 7 commits into
vectorize-io:mainfrom
TheAngryPit:codex/coding-agents-survey-document-routing
Open

fix(coding-agents): route repository surveys explicitly#3343
TheAngryPit wants to merge 7 commits into
vectorize-io:mainfrom
TheAngryPit:codex/coding-agents-survey-document-routing

Conversation

@TheAngryPit

@TheAngryPit TheAngryPit commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Route the four canonical repository-survey documents through the survey strategy instead of generic document ingestion.
  • Attach deterministic source:survey, harness, retain-attribution, and structural knowledge tags.
  • Ground Component map, Core concepts, and Conventions page queries in their canonical survey evidence using repository-generic wording.
  • Make sync status, crashed-survey retry detection, and completion-marker handling recognize source:survey while retaining compatibility with legacy source:upload banks.
  • Keep the router's document IDs compile-time aligned with the survey producer's canonical SURVEY_DOC_IDS contract.

Root cause

hindsight_ingest_document treats every title as a generic uploaded document. The repository survey uses that public tool for four fixed findings, so vanilla ingestion classifies them as document / source:upload, while the seeded Knowledge Pages expect deterministic structural tags. Page population therefore depends on incidental extraction instead of the native survey contract.

Current-main refresh

This branch was rebuilt on current main (64ede361) instead of mechanically preserving the historical implementation. It keeps the current mission/template ownership model, current retain attribution, and the current client contract. It does not restore removed retain options or unrelated historical template changes.

Validation

Focused survey/router/readers
  104 tests passed

Full Coding Agents suite
  45 files passed, 493 tests passed

TypeScript --noEmit
  passed

Production build and declarations
  passed

Prettier and git diff --check
  passed

Independent review
  safe to merge; no blockers

No live Hindsight retain or page-refresh mutation was performed during this refresh. The earlier local data remediation is not part of this source PR.

Compatibility

  • Non-survey hindsight_ingest_document calls retain document / source:upload behavior.
  • Existing banks whose survey findings use source:upload remain supported.
  • Existing user-owned missions are not overwritten.
  • No server API, database schema, migration, dependency, or hook-configuration change.

@TheAngryPit

Copy link
Copy Markdown
Contributor Author

Updated onto current main at 3a4b021 and resolved the retain-options conflict introduced by #3336: survey retains now use the current always-async client contract without the removed async option. I also rewrote the three seeded page queries to stay repository-generic while retaining deterministic anchors to the canonical survey documents; local repair, remediation, tag-repair, and harness-diagnosis wording is no longer part of the upstream queries. Fresh validation: focused routing/page/status/session tests 81/81; full Coding Agents suite 40 files and 412 tests; TypeScript noEmit and production build passed; diff check clean.

@TheAngryPit
TheAngryPit force-pushed the codex/coding-agents-survey-document-routing branch from 3569efa to 9d05f5a Compare August 12, 2026 11:18
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