Skip to content

docs(readme): reflect PR #886 npm B1 landing in Epic 1 status#889

Merged
s977043 merged 1 commit into
mainfrom
docs/readme-en-800-drift
May 24, 2026
Merged

docs(readme): reflect PR #886 npm B1 landing in Epic 1 status#889
s977043 merged 1 commit into
mainfrom
docs/readme-en-800-drift

Conversation

@s977043

@s977043 s977043 commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

Codex's post-25-PR final-pass review flagged this drift: both README.md and README.en.md's Roadmap table still show Epic 1 (First-Run Adoption / #800) as "Planned — not started" despite PR #886 landing the B1 deliverable (npm distribution design + dry-run packaging audit).

Change

Update Epic 1's "状態 / Status" cell on both READMEs to read:

Partial — B1 design + dry-run packaging audit landed (#886). B2 (package.json implementation + publish verification) not yet (#800)

Mirrored 1:1 between JA and EN versions.

Rationale

Codex (final-pass search after the npm audit fix in #888) returned three drift candidates:

  1. README Epic 1 status (this PR)
  2. .river/risk-map.yaml example missing from examples/
  3. release smoke runbook not yet under docs/runbook/

Per Codex's "1 PR 1 論点 / 連続セッション後は typo / 整合漏れリスクが高い" guidance, only #1 is addressed here. #2 and #3 stay open for future small slices.

Test plan

  • markdownlint + prettier green
  • node scripts/fix-dashes.mjs --check clean
  • CI green

🤖 Generated with Claude Code

Codex's final-pass review (after 25 PRs + Codex stop) flagged this
drift: the README Roadmap table on both README.md and README.en.md
still reads "Planned — not started" for Epic 1, but PR #886 landed
the B1 deliverable (#800 npm distribution design + dry-run packaging
audit). Update both files to "Partial" with concrete pointers:

- B1 (design + audit) landed in #886.
- B2 (package.json implementation + publish verification) remains
  open under #800.

No content change beyond the status cell; the rest of the legend
(Implemented / Partial / Planned) stays canonical.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@vercel

vercel Bot commented May 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ai-review-kit Ready Ready Preview, Comment May 24, 2026 6:36am

@github-actions

Copy link
Copy Markdown
Contributor

No changes to review compared to main.

@github-actions

Copy link
Copy Markdown
Contributor

PlanGate Review

  • decision: pass
  • plan status: skipped-by-label
  • exec status: skipped-by-label

PlanGate review decision: pass

severity count policy
critical 0 fail
major 0 fail-if-required / warn
minor 0 comment-only
info 0 skipped

ポリシー: critical=fail / major=fail-if-required (warn) / minor=comment-only / info=skipped — spec: pages/reference/cli-review-plan-spec.md

@s977043 s977043 merged commit 59f879b into main May 24, 2026
18 checks passed
@s977043 s977043 deleted the docs/readme-en-800-drift branch May 24, 2026 06:38
s977043 added a commit that referenced this pull request May 25, 2026
* docs(runbook): add release-smoke.md (codex final-pass review item #3)

Close the third drift item from Codex's final-pass review (after the
v0.51.0–v0.55.0 sprint). Items #1 and #2 already landed:
- #1 README Epic 1 status drift (PR #889)
- #2 risk-map example (PR #896)
- #3 release smoke runbook (this PR)

Add docs/runbook/release-smoke.md as a sibling to docs/runbook/dev.md.
It is not a release runbook (release-please handles tagging), but the
smoke test that confirms a fresh tag installs cleanly for an adopter.

Sections
- When to run: post release-please merge, post dependabot prod-dep
  bump, pre announcement.
- What it verifies: lint/tests/skills validate, river run dry-run,
  npm pack --dry-run, npm audit --omit=dev, GitHub Release presence.
- Smoke sequence: single bash block, all commands must exit 0.
- Release confirmation via gh: tagName / publishedAt / changelog body.
- When the smoke fails: concrete recovery moves for the 5 common
  failure modes (lockfile drift, dist staleness, audit vuln, release
  404, skill loader error), pointing at the existing docs and PRs
  that document each fix.
- Out of scope: cross-platform install, npx river try, real LLM, Vercel.

No code change.

Refs: PRs #889, #896 (sibling Codex final-pass items),
docs/development/npm-distribution-design.md, AGENT_LEARNINGS.md
gh-api empty-commit entry.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>

* docs(runbook): apply gemini PR #897 review on release-smoke

Both gemini comments are valid; apply.

1. Resolve TAG via the upstream GitHub Release instead of the local
   package.json. The previous form trusted that the operator had
   already pulled main after the release-please merge; if they had
   not, the local package.json could point at the previous
   version and the smoke would run against the wrong commit. Use
   `gh release view --json tagName --jq .tagName` so the upstream
   release is the source of truth.

2. The "feedback_release_please_workflow_race.md memory if available"
   pointer was misleading — that file is Claude's internal memory,
   not a repo doc. Replace with a concrete pointer to the
   AGENT_LEARNINGS.md 2026-05-24 entry that actually carries the
   gh-api empty-commit pattern in the repo.

No content change beyond these two clarifications.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <[email protected]>
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