Skip to content

README: list the handoff-needs-smoke-test hook - #535

Merged
mergify[bot] merged 1 commit into
mainfrom
stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d
Sep 14, 2026
Merged

README: list the handoff-needs-smoke-test hook#535
mergify[bot] merged 1 commit into
mainfrom
stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Sep 13, 2026

Copy link
Copy Markdown
Owner

Summary

The main readme has a table listing every safety hook and what it does.

This adds the row for the new hook lower in this stack. That hook checks that a script handed to a person has been run first.

The PR checker counts the main readme as documentation, a separate kind of change, so the row lives in its own PR.

Review Claim

Approve one new row in the main readme's hook table, for the hook added at the bottom of this stack.

Review Lane

docs

Review Unit

docs

Safety Invariant

Only one table row is added. No code changes.

Slice Rationale

The row describes a hook added lower in this stack, so this lands after it. Keeping it separate lets the hook PR hold one kind of change.

Non-goals

  • Does not change the hook, its own guide, or its inventory row.
  • Does not list the login-shell hook; that hook is not in this stack.

Test Plan

Test Plan

Every CI command, run on this commit:

rc=0  bash scripts/run_all_tests.sh
rc=0  python3 scripts/check_no_tracked_local_artifacts.py
rc=0  python3 scripts/check_hook_test_coverage.py
rc=0  python3 scripts/check_rules_are_wired.py
rc=0  python3 scripts/check_skills_three_harnesses.py
rc=0  python3 scripts/check_ecosystem_boundaries.py
rc=0  python3 scripts/check_skill_file_refs.py
rc=0  python3 scripts/check_skill_test_coverage.py --base origin/main --head HEAD
rc=0  python3 scripts/check_skill_test_debt_no_growth.py
rc=0  python3 scripts/check_skill_trigger_mechanism.py
rc=0  python3 scripts/check_dora_baseline.py
rc=0  python3 scripts/check_no_dated_provenance.py --base origin/main
rc=0  python3 scripts/check_no_new_comments.py --base origin/main
rc=0  shellcheck install.sh

Revert Plan

Revert Plan
  • Safe to revert? Yes
  • Revert command: git revert <sha>
  • Post-revert steps: None. The row leaves the readme table.
  • Data migration? No

🤖 Generated with Claude Code

Depends-On: #511


Note

Low Risk
Documentation-only: one new table row in README with no code or behavior changes.

Overview
Documents the handoff-needs-smoke-test safety hook in the main README hooks table with a new row placed after ui-input-guard.

The row states that when a reply hands the user a script via ! bash <path> that was never run in the session, the agent must run it or explain why it cannot run here.

Reviewed by Cursor Bugbot for commit 80028fe. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor

cursor Bot commented Sep 13, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_6980cd14-9a4a-4131-a5b6-715c8daa7ad5)

@EdbertChan

EdbertChan commented Sep 13, 2026

Copy link
Copy Markdown
Owner Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 hook: a script handed to the user is a claim that it runs #506
2 audit(token_audit): count "bogus" and "it didn't run" as the same class #507
3 gate: a branch taken per operating system needs a test that injects one #509
4 audit(token_audit): one submission carrying two slash commands is one message #510
5 audit: use the transcript's own markers for what the human actually sent #511
6 README: list the two new hooks from this stack #535 👈

@EdbertChan EdbertChan changed the title README: list the handoff-needs-smoke-test and remote-payload-collapses hooks README: list the two new hooks from this stack Sep 13, 2026
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c branch from 4894e27 to e1ac807 Compare September 13, 2026 18:39
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from 062b3db to 5c0e9dc Compare September 13, 2026 18:39
@EdbertChan EdbertChan changed the title README: list the two new hooks from this stack README: list the handoff-needs-smoke-test hook Sep 13, 2026
@cursor

cursor Bot commented Sep 13, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_e5f100c5-e9ef-4e81-9d56-f1151354611b)

@EdbertChan

Copy link
Copy Markdown
Owner Author

Revision history

# Type Changes Reason Date
1 initial 062b3db 2026-09-13 18:39 UTC
2 content 062b3db → 5c0e9dc (raw) 2026-09-13 18:39 UTC

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c branch from e1ac807 to b6d9209 Compare September 14, 2026 04:16
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from 5c0e9dc to e8a3557 Compare September 14, 2026 04:18
@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_cf226e98-0dd2-4b86-b300-b8f7785835e9)

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from e8a3557 to 9f832e0 Compare September 14, 2026 04:34
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c branch from b6d9209 to 0da72ef Compare September 14, 2026 04:34
@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_62320c7e-c0d8-40e3-8a6e-8cec9df9e83a)

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from 9f832e0 to bf54339 Compare September 14, 2026 05:14
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c branch from 0da72ef to 87c0adb Compare September 14, 2026 05:14
@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_a49484af-fb4b-422c-b066-afd0bfd4a75d)

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c branch from 87c0adb to 38b78bc Compare September 14, 2026 05:27
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from bf54339 to e19314a Compare September 14, 2026 05:27
@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_87b4af01-bdaf-48e4-85cf-d9e4e73c4c25)

Base automatically changed from stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/use-transcript-s-own-markers-what-human-actually--a2a6527c to main September 14, 2026 05:38
Adds the hook's row to the README hook table. The row moved out of the
hook PR because the PR checker counts the root README as its own docs
unit; the hook's own README and inventory row stay with the hook.

Change-Id: I0618d41d327180cc18b1b8daf4820706f912ffc1
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch from e19314a to 80028fe Compare September 14, 2026 05:38
@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_7d23ad88-d457-4922-94fd-70b7a4e1a298)

@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 6 minutes 1 second in the queue, including 5 minutes 21 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Sep 14, 2026
@mergify
mergify Bot merged commit 8f032dd into main Sep 14, 2026
6 checks passed
@mergify
mergify Bot deleted the stack/EdbertChan/reflect/ui-input-guard-hook-freshness-20260908/list-handoff-needs-smoke-test-remote-payload--0618d41d branch September 14, 2026 05:49
@mergify mergify Bot removed the queued label Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant