Skip to content

Days 1–5: application comparison baseline, scope design and roadmap - #869

Merged
pengfei-threemoonslab merged 3 commits into
mainfrom
codex/days1-5-roadmap-evidence
Sep 27, 2026
Merged

pengfei-threemoonslab merged 3 commits into
mainfrom
codex/days1-5-roadmap-evidence

Conversation

@pengfei-threemoonslab

@pengfei-threemoonslab pengfei-threemoonslab commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Scope

Days 1–5 of the owner-approved roadmap. Refs #778, #868, #580, #655, #610, #795, #812, #780, #369. These implementation epics remain open.

Engineering review

Two coding-agent self-review/address rounds, documented in docs/research/application-days1-5/review.md. Corrected verifier-artifact projection, truncated error data and scan/verify denominator ambiguity. No remaining blocking design findings. This is not independent human qualification.

Validation

  • Ruff: pass. git diff --check did not pass at 8a0dbbd: README.md had a trailing space and replay.csv used CRLF line endings. Both were fixed in 523725f, and it passes now.
  • CI recipe / adoption pin targeted tests: 16 passed.
  • Full hosted CI passed on 8a0dbbd: all three suite shards, core test job, Windows/clean-checkout launchers, both MCP dependency jobs and both verifier checks. Release-tag consistency is intentionally skipped on PRs. The first local run failed at sandboxed wheel-build setup; the network-enabled duplicate local run was stopped after hosted full validation passed. The local run also exposed fixture commits blocked by the machine-wide Git ignore rule for .claude/settings.local.json; all 41 tests in that file pass with core.excludesFile=/dev/null scoped to the test process. User Git settings were not changed. No local full-suite pass is claimed.
  • Documentation/research change only; no manifest, policy, instruction or runtime implementation edits.

Review follow-up (2026-09-27)

Review fixes are in 523725f:

Review and fixes by a coding agent (Claude Code), not an independent human approval.

@pengfei-threemoonslab

Copy link
Copy Markdown
Contributor Author

Days 1–5 validation closeout

The two coding-agent review/address rounds are recorded in this PR. Full hosted CI passed on the published head; the release-tag-only check is intentionally skipped on PRs. The PR is ready for maintainer review.

Local validation: lint passes; the initial sandboxed wheel-build setup needed network. A separate fixture failure was traced to the existing global Git ignore rule for .claude/settings.local.json; all 41 tests in that file pass when only core.excludesFile is isolated to /dev/null for that process. No user Git setting was changed and no local full-suite pass is claimed.

#869 records the baseline and design; #870 delivers #787's bounded repair with 14 focused tests and a current, complete verifier result. #610 remains a reproduced contract defect; #580/#655/#867 implementation and the ten-case proof remain in the next phase. Selected issues and milestones have been reconciled without closing those outstanding acceptance criteria.

The ROADMAP "Selected execution" section and the research README's
issue table describe as future work items that have since merged: #870
closed #787, #871 merged diff --application, #873 closed #580 and #879
closed #864. Add a dated "Status as of 2026-09-27" note to each naming
what merged and what remains open, leaving the plan and snapshot as
written. comparison-design.md called itself the selected design,
delivered through the existing capability projection and "not a
promised new CLI flag", but #871 added a new flag with its own advisory
JSON schema; add a status banner saying it is superseded in part.

Renaming "Lead wedge (focus)" broke the inbound link from
docs/engineering/v1-release-readiness.md to #lead-wedge-focus; keep the
old slug with an anchor above the renamed heading, and do the same for
the renamed post-1.0 adoption heading.

git diff --check failed on a trailing space in the research README and
CRLF endings on every line of replay.csv. Strip the space and convert
the CSV to LF; nothing pins its digest and no field holds a newline.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pengfei-threemoonslab
pengfei-threemoonslab merged commit 36c061d into main Sep 27, 2026
12 checks passed
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