docs: say a dispatch run needs the pr_number input #43

Merged

jercik merged 1 commit from docs/dispatch-pr-number-input into main

2026-10-05 09:34:56 +00:00

jercik commented

2026-10-05 00:16:43 +00:00

Owner

Follow-up to #37, from this finding: the README never said a workflow_dispatch run needs the calling workflow to declare a pr_number input. It targets main because the sentence it adds sits in a paragraph #37 doesn't touch.

🤖 Generated with Claude Code

Follow-up to #37, from [this finding](https://code.j4k.dev/j4k-oss/review-wrapper/pulls/37#issuecomment-125248): the README never said a `workflow_dispatch` run needs the calling workflow to declare a `pr_number` input. It targets `main` because the sentence it adds sits in a paragraph #37 doesn't touch. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

jercik added 1 commit

2026-10-05 00:16:43 +00:00

docs: say a dispatch run needs the pr_number input

commit-msg / commitlint (pull_request) Successful in 27s

Details

Checks / quality-checks (pull_request) Successful in 1m1s

Details

Review / Review (pull_request_target) Successful in 2m49s

Details

53b7dece47

The dispatch arm of `readWrapperInputs` reads the pull request's number
from the `pr_number` input and throws when it is absent. The README never
named that input.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

jercik referenced this pull request

2026-10-05 00:18:12 +00:00

docs: correct the service URL and PR-input rows of the environment table #37

jercik referenced this pull request

2026-10-05 00:18:17 +00:00

docs: correct the service URL and PR-input rows of the environment table #37

forgejo-actions commented

2026-10-05 00:19:32 +00:00

Review 01M44PQ3TDP05TF84GMFVTJHEA — head 53b7dece47085f8163d77930b7d138109d464cf7

Review — j4k-oss/review-wrapper @ `02167c099a`

Scope: diff against base tree e69943819640
Status: dispatched — coverage complete (5/5 slots terminal)
Facts: current review-wide projection

Computed under:

{
  "abandonment": "abandonment-v1",
  "anchor_recipe": 1,
  "batch_policy": "batch-v1",
  "coverage": "coverage-v3",
  "dispatch_policy": "dispatch-v2",
  "grounder_version": 1,
  "grounding_read_rule": "grounding-read-v1",
  "promotion_policy": "promotion-v1",
  "report": "report-v4",
  "tally": "tally-v1",
  "triage_settle": "triage-settle-v2"
}

Findings (0)

No findings survived.

Reviewed:

general-bug (whole/default): no-claims
writing-quality (whole/default): no-claims
test-trimming (whole/default): no-claims
restated-sets (whole/default): no-claims
project-docs (whole/default): no-claims

Other claims

grounding-pending (0)
ungrounded (0)
rejected (0)
duplicate-of (0)
unadjudicated (0)

Coverage

Coverage pass: 01M44PQ3VZV88AXDBE1KT45VEN
Accounting: complete
Slot health: healthy

lens	part	arm	unit status	runs	loss
general-bug	whole	default	no-claims	1	no
writing-quality	whole	default	no-claims	1	no
test-trimming	whole	default	no-claims	1	no
restated-sets	whole	default	no-claims	1	no
project-docs	whole	default	no-claims	1	no

**Review** `01M44PQ3TDP05TF84GMFVTJHEA` — head `53b7dece47085f8163d77930b7d138109d464cf7` # Review — j4k-oss/review-wrapper @ 02167c099a79 Scope: diff against base tree `e69943819640` Status: dispatched — coverage complete (5/5 slots terminal) Facts: current review-wide projection Computed under: ```json { "abandonment": "abandonment-v1", "anchor_recipe": 1, "batch_policy": "batch-v1", "coverage": "coverage-v3", "dispatch_policy": "dispatch-v2", "grounder_version": 1, "grounding_read_rule": "grounding-read-v1", "promotion_policy": "promotion-v1", "report": "report-v4", "tally": "tally-v1", "triage_settle": "triage-settle-v2" } ``` ## Findings (0) No findings survived. Reviewed: - general-bug (whole/default): no-claims - writing-quality (whole/default): no-claims - test-trimming (whole/default): no-claims - restated-sets (whole/default): no-claims - project-docs (whole/default): no-claims ## Other claims - grounding-pending (0) - ungrounded (0) - rejected (0) - duplicate-of (0) - unadjudicated (0) ## Coverage Coverage pass: 01M44PQ3VZV88AXDBE1KT45VEN Accounting: complete Slot health: healthy | lens | part | arm | unit status | runs | loss | | --- | --- | --- | --- | --- | --- | | general-bug | whole | default | no-claims | 1 | no | | writing-quality | whole | default | no-claims | 1 | no | | test-trimming | whole | default | no-claims | 1 | no | | restated-sets | whole | default | no-claims | 1 | no | | project-docs | whole | default | no-claims | 1 | no |

🚀 1

jercik merged commit 4427f01de8 into main

2026-10-05 09:34:56 +00:00

jercik referenced this pull request from a commit

2026-10-05 09:34:56 +00:00

docs: say a dispatch run needs the pr_number input (#43)