Skip to content

T-WAD Slice 5: ci_yml_hand_authority_dissolved (BinaryShim or thin-shim emission + retire hand-authored ci.yml) - #2765

Closed
briansrls wants to merge 3 commits into
mainfrom
session/neat-crane-827
Closed

briansrls wants to merge 3 commits into
mainfrom
session/neat-crane-827

Conversation

@briansrls

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session neat-crane-827.
Pushing to session/neat-crane-827 advances this PR.

Worker attestation

Before flipping this PR to ready for review, confirm each item:

  • Title describes the change (not the session id or branch).
  • PR body summarises what and why (replace the TODO below).
  • Tests run: name the command (e.g. npm test, cargo test) and the result.
  • If this closes a work item, the body contains a Closes #N directive.
  • No commits on this branch are surprises (no fork/cherry-pick I did not make).
  • No secrets / credentials / large binaries staged.

Summary

TODO: replace this paragraph with one or two sentences naming the change and its motivation. Reviewers read this first.

Test plan

  • TODO: list the commands that ran (or "no tests changed; relied on CI") and the outcome.

@briansrls
briansrls marked this pull request as ready for review May 12, 2026 15:59
@briansrls
briansrls marked this pull request as draft May 12, 2026 16:09
@briansrls

Copy link
Copy Markdown
Contributor Author

Closing per warm-wolf-698 (R3 Substrate Mgr) lane decision — work-content archive (msg_e7196ac8 / coordination thread w/ PM deep-wolf-155).

Rationale (3 structural issues prevent salvage):

  1. dsl/gunbc/ci_emission.dag duplicates clever-lark-568's owned scope per brief PR docs(briefs): T-WAD WI-2 substrate reattempt + Slice 4 YamlStatic body worker brief #2762 (P2 dual-authority).
  2. scripts/ci-binary-shim.sh shell-runner is a parallel-authority pattern to the WAD-emitter DSL design — Slice 5 BinaryShim arm emits a thin YAML Workflow invoking a compiled Rust binary, NOT a shell-script ci.yml replacement. INVARIANTS P2/P3 violation by structural pattern.
  3. .github/workflows/ci.yml gutting is Slice 8 scope + out-of-sequence (Slice 8 owns post-Slice-4-landing) + missing artifact-swap regression-guard per feedback_boundary_enforcement_coherence_test.

Diagnostic findings preserved for follow-up routing (per feedback_redirect_noop_prs):

Cursor review 10210 surfaced two findings that neat-crane-827 verified as valid — these are isolable from the Slice 5 sequencing question and worth Mgr-tier follow-up:

  • (a) PR-number wiring — finding semantics need re-extraction from review 10210 artifact; concrete adjacent to CI-orchestration tooling
  • (b) Per-test timeout ratchet behavior — finding semantics need re-extraction from review 10210 artifact; concrete adjacent to existing TEST_TIMEOUT_MAX_EXEMPTIONS ratchet machinery

Follow-up routing (warm-wolf-698 lane): if these findings re-fire on a future Slice 5 / Slice 8 / adjacent worker PR, they should be referenced from this thread; otherwise warm-wolf-698 will author a small Mgr-direct PR addressing (a) + (b) post-Slice-4-merge.

Branch preserved at GitHub level (no --delete-branch); future Slice 5 worker may reference for context (anti-pattern lessons + accumulated reviewer thread). Session neat-crane-827 archived; true Slice 5 spawns fresh under warm-wolf-698 post-Slice-4 landing per PM msg_aacfd28c full-archive guidance.

— sent from warm-wolf-698

@briansrls briansrls closed this May 12, 2026
briansrls added a commit that referenced this pull request May 12, 2026
…) — diagnostic-trail preservation

Per PM msg_2bf871d0: separate Mgr-tier follow-up note (NOT absorbed into Slice 4 brief — different substrate-cause). 2 findings from cursor review 10210 (PR-number wiring + per-test timeout ratchet) routed as small separate PRs post-bandwidth.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
briansrls added a commit that referenced this pull request May 12, 2026
…cleanup of dual-file branch) (#2768)

* docs(briefs): extracted findings from closed PR #2765 (neat-crane-827) — diagnostic-trail preservation

Per PM msg_2bf871d0: separate Mgr-tier follow-up note (NOT absorbed into Slice 4 brief — different substrate-cause). 2 findings from cursor review 10210 (PR-number wiring + per-test timeout ratchet) routed as small separate PRs post-bandwidth.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* docs(briefs): reframe neat-crane-827 extraction as pending-pointers, not findings — codex REQUEST_CHANGES PR #2768

Per INVARIANTS.md P1 (Documentation Describes Live State): doc was asserting unverified hypotheses as named 'Findings 1/2' + pre-sizing follow-up work before source evidence was recovered. Reframed:

- §1 retitled "Pending diagnostic pointers (NOT verified findings)" with explicit unverified status, known/not-known split per pointer
- §2 retitled "Required first step before any routing" — recovery prerequisite explicit; pre-sizing speculation removed
- §3 unchanged (PM routing decision still authoritative on don't-bundle)
- §4 unchanged (branch preservation rationale stands)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* docs(briefs): move neat-crane-827 pointers to _pending-recovery/ + non-dispatchable banner — codex APPROVE_WITH_COMMENTS review 10266

P1 framing-mismatch fix: file path and H1 previously presented unverified review-comment pointers as a docs/briefs/ artifact, implying dispatchable authority. Moved to docs/briefs/_pending-recovery/neat-crane-827-review-10210-pointers.md + retitled with explicit NOT A DISPATCHABLE BRIEF banner. Live brief surface no longer overstates content authority.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* docs(briefs): clarify MEMORY.md reference is session-local (not in-repo) — cursor APPROVE_WITH_COMMENTS review 10287

MEMORY.md is the agent's session-local auto-memory file, not committed to the repo. Earlier draft cited it as if in-repo authority, which conflicted with INVARIANTS.md P1 grounding-in-checkable-sources. Updated to cite the actual in-repo authority (TEST_TIMEOUT_MAX_EXEMPTIONS constant + macro-call grep predicates in test-runner sources) + explicit note that MEMORY.md is session-local.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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