Skip to content

fix(pi): accept Architect records without deprecated mode - #11

Merged
dougyau merged 1 commit into
mainfrom
fix/pi-architecture-without-mode
Oct 20, 2026
Merged

dougyau merged 1 commit into
mainfrom
fix/pi-architecture-without-mode

Conversation

@dougyau

@dougyau dougyau commented Oct 20, 2026

Copy link
Copy Markdown
Owner

Pitako pingdotgg#56 intentionally stopped emitting mode for new Architect tables. T3 still required it, so the authenticated bridge rejected valid replies and the table never reached the archive.

Update the passive protocol snapshot and carry the optional legacy field through validation, ingestion, queries, and the shared client model without inventing a default. Keep historical records readable, reject malformed current compositions, and preserve composition reasons and member mandates.

Exclude the two verbatim protocol copies from formatting so the normal commit hook preserves their upstream bytes.

Verification

  • 230 focused tests pass across six files, including two real native CLI tests with offline providers.
  • Scoped lint, configuration formatting, and diff checks pass. Independent review of the complete 16-file diff found no blockers.
  • Six affected compiler subprocesses returned zero. Server typechecking still reports five existing cleanup TS2339 errors. The enclosing compiler tool failed with owned process group remained after close; this is not an all-green typecheck result.
  • No installed rebuild or positive live-archive or visual verification is claimed.

Model: GPT-6.1 Sol. Harness: Pi with Pitako. Independent review: GPT-6 Sol.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Oct 20, 2026
@dougyau
dougyau merged commit 2d18b76 into main Oct 20, 2026
7 of 22 checks passed
@dougyau
dougyau deleted the fix/pi-architecture-without-mode branch October 20, 2026 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant