Skip to content

docs(ship-spec): lessons from the #989 run - #1000

Merged
Daanish2003 merged 1 commit into
mainfrom
docs/989-ship-spec-lessons
Oct 29, 2026
Merged

Daanish2003 merged 1 commit into
mainfrom
docs/989-ship-spec-lessons

Conversation

@Daanish2003

Copy link
Copy Markdown
Owner

Summary

Lessons from the /ship-spec 989 run (Approval rules, PR #999), added to .claude/skills/ship-spec/environment.md:

  • The lockfile-only and snapshot-only hook bullet now records the owner's lint-staged fix (9f54d70).
  • A new migration must also update the migrate-cli --print snapshot.
  • A reviewer timeout fails the real safety eval; re-run with EVAL_REPEATS=1.
  • The integration worktree folder was EPERM-locked after the merge.

How it was tested

Docs only; the pre-commit hook (oxlint and oxfmt) passed.

Decisions I made

none

Risk

none

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Daanish2003
Daanish2003 merged commit 469a7e4 into main Oct 29, 2026
1 of 2 checks passed
@Daanish2003
Daanish2003 deleted the docs/989-ship-spec-lessons branch October 29, 2026 15:37
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