@joshuafolkken/kit 1.1050.0 → 1.1094.0

package/scripts/issue/issue-backlinks-cli.ts

@@ -1,6 +1,6 @@
 #!/usr/bin/env tsx
 import { fileURLToPath } from 'node:url'
-import { git_gh_command } from '#scripts/git/git-gh-command'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { parse_json_object_safe } from '#scripts/git/parse-json-array'
 import { z } from 'zod'
 import { issue_backlinks, type UpstreamEntry } from './issue-backlinks'
@@ -90,8 +90,4 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv)
 }
 
-const issue_backlinks_cli = { run, USAGE }
-
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))
-
-export { issue_backlinks_cli }

package/scripts/issue/issue-cite-cli.ts

@@ -1,8 +1,8 @@
 #!/usr/bin/env tsx
 import { fileURLToPath } from 'node:url'
 import { parseArgs } from 'node:util'
-import { git_gh_command } from '#scripts/git/git-gh-command'
-import type { IssueRead } from '#scripts/git/git-gh-issue-read'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
+import type { IssueRead } from '#scripts/gh/git-gh-issue-read'
 import { bounded_pool } from '#scripts/lib/bounded-pool'
 import { z } from 'zod'
 import { issue_cite, type CiteTarget } from './issue-cite'
@@ -145,12 +145,8 @@ async function main(args: ReadonlyArray<string>): Promise<void> {
 
 const issue_cite_cli = {
 	USAGE,
-	main,
 	parse_targets,
-	parse_title,
-	report,
 	run,
-	to_line,
 }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))

package/scripts/issue/issue-cite.ts

@@ -105,11 +105,9 @@ function no_repo_line(target: CiteTarget): string {
 }
 
 const issue_cite = {
-	SUMMARY_SEPARATOR,
 	citation_line,
 	citer,
 	issue_url,
-	label,
 	missing_line,
 	no_repo_line,
 	parse_target,

package/scripts/issue/issue-classification.ts

@@ -1,10 +1,5 @@
-import {
-	BREAKING_CHANGE_LABEL,
-	BUG_LABEL,
-	ENHANCEMENT_LABEL,
-	has_label_name,
-} from '#scripts/git/issue-labels'
 import { issue_bug_label } from './issue-bug-label'
+import { BREAKING_CHANGE_LABEL, BUG_LABEL, ENHANCEMENT_LABEL, has_label_name } from './issue-labels'
 import { markdown_section } from './markdown-section'
 
 const FEATURE_DECLARATIONS = ['- 目的: 機能追加', '- 目的: 機能改善']
@@ -62,10 +57,7 @@ function problems(body: string): ReadonlyArray<string> {
 }
 
 const issue_classification = {
-	BREAKING_DECLARATION,
 	CLASSIFICATION_LABELS,
-	FEATURE_DECLARATIONS,
-	NON_BUG_DECLARATION,
 	labels_for,
 	problems,
 	required_labels,

package/scripts/issue/issue-closing-pr.ts

@@ -1,7 +1,7 @@
+import { git_gh_api_path } from '#scripts/gh/git-gh-api-path'
+import { git_gh_exec } from '#scripts/gh/git-gh-exec'
+import { git_gh_repo } from '#scripts/gh/git-gh-repo'
 import { git_closes_keyword } from '#scripts/git/git-closes-keyword'
-import { git_gh_api_path } from '#scripts/git/git-gh-api-path'
-import { git_gh_exec } from '#scripts/git/git-gh-exec'
-import { git_gh_repo } from '#scripts/git/git-gh-repo'
 import { parse_json_object_safe } from '#scripts/git/parse-json-array'
 import { z } from 'zod'
 
@@ -88,6 +88,6 @@ async function read_closing_pr(issue: string): Promise<string | undefined> {
 	}
 }
 
-const issue_closing_pr = { parse_references, pick_closing_pr, read_closing_pr }
+const issue_closing_pr = { read_closing_pr }
 
 export { issue_closing_pr }

package/scripts/issue/issue-comment-cli.ts

@@ -1,6 +1,6 @@
 #!/usr/bin/env tsx
 import { fileURLToPath } from 'node:url'
-import { git_gh_command } from '#scripts/git/git-gh-command'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { cli_body } from '#scripts/josh/cli-body'
 import { error_text } from '#scripts/lib/error-message'
 
@@ -89,7 +89,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv)
 }
 
-const issue_comment_cli = { USAGE, main, parse_request, run }
+const issue_comment_cli = { USAGE, run }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))
 

package/scripts/issue/issue-file-cli.ts

@@ -3,10 +3,10 @@ import { readFile } from 'node:fs/promises'
 import { fileURLToPath } from 'node:url'
 import { repo_party } from '#scripts/discovery/repo-party'
 import { epic_bundle_cli } from '#scripts/epic/epic-bundle-cli'
-import { git_gh_command } from '#scripts/git/git-gh-command'
-import { git_gh_exec } from '#scripts/git/git-gh-exec'
-import { git_gh_issue_write } from '#scripts/git/git-gh-issue-write'
-import { github_issue_url } from '#scripts/git/github-issue-url'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
+import { git_gh_exec } from '#scripts/gh/git-gh-exec'
+import { git_gh_issue_write } from '#scripts/gh/git-gh-issue-write'
+import { github_issue_url } from '#scripts/gh/github-issue-url'
 import { error_text } from '#scripts/lib/error-message'
 import { issue_file, type FileArguments } from './issue-file'
 import { issue_lint_cli } from './issue-lint-cli'
@@ -182,7 +182,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv)
 }
 
-const issue_file_cli = { USAGE, THIRD_PARTY_MESSAGE, UNKNOWN_REPO_MESSAGE, run, main }
+const issue_file_cli = { THIRD_PARTY_MESSAGE, run }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))
 

package/scripts/issue/issue-file.ts

@@ -1,9 +1,9 @@
 import { parseArgs } from 'node:util'
 import { epic_issue } from '#scripts/epic/epic-issue'
-import { github_issue_url } from '#scripts/git/github-issue-url'
-import { DEPTH_LABEL_ORDER, FILING_ROUTE_LABELS, has_label_name } from '#scripts/git/issue-labels'
+import { github_issue_url } from '#scripts/gh/github-issue-url'
 import { issue_backlinks } from './issue-backlinks'
 import { issue_classification } from './issue-classification'
+import { DEPTH_LABEL_ORDER, FILING_ROUTE_LABELS, has_label_name } from './issue-labels'
 import { markdown_section } from './markdown-section'
 
 // The decisions behind `josh issue:file` (joshuafolkken/kit#2808), kept apart from the command that

package/scripts/issue/issue-fold-cli.ts

@@ -90,7 +90,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv)
 }
 
-const issue_fold_cli = { JSON_KEY, USAGE, REASONS, read_arguments, size_verdict, run, main }
+const issue_fold_cli = { read_arguments, size_verdict, run }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))
 

package/scripts/issue/issue-fold-existing-cli.ts

@@ -74,7 +74,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv)
 }
 
-const issue_fold_existing_cli = { run, main, USAGE }
+const issue_fold_existing_cli = { run }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))
 

package/scripts/{git → issue}/issue-labels.ts

@@ -6,6 +6,11 @@ const BUG_LABEL = 'bug'
 const ENHANCEMENT_LABEL = 'enhancement'
 const BREAKING_CHANGE_LABEL = 'breaking-change'
 const IN_PROGRESS_LABEL = 'in-progress'
+// Applied by `kickoff` once its plan is on the issue, so the next command's `run:entry` reads the issue
+// as planned and starts at the implementation rather than planning again (joshuafolkken/kit#3042).
+// Namespaced like `run:solo`: a bare `planned` is a common roadmap label a person may already apply,
+// and reading one as kickoff's mark would skip the planning that never happened.
+const PLANNED_LABEL = 'run:planned'
 // Parks a child that cannot advance without a person deciding something. `epic:next` is what reads
 // it: a parked child is why a run reports "nothing left that time will fix" rather than waiting
 // forever (joshuafolkken/kit#860).
@@ -207,6 +212,7 @@ const WORKFLOW_LABELS: ReadonlyArray<LabelDefinition> = [
 		description: 'Tracks a batch of child issues from one split',
 	},
 	{ name: IN_PROGRESS_LABEL, color: '0075ca', description: 'Work is actively in progress' },
+	{ name: PLANNED_LABEL, color: 'c2e0c6', description: 'kickoff posted the plan' },
 	{
 		name: RUN_SOLO_LABEL,
 		color: 'b60205',
@@ -328,6 +334,7 @@ const ALL_LABELS: ReadonlySet<string> = new Set([
 	ENHANCEMENT_LABEL,
 	BREAKING_CHANGE_LABEL,
 	IN_PROGRESS_LABEL,
+	PLANNED_LABEL,
 	NEEDS_DECISION_LABEL,
 	AUTO_OK_LABEL,
 	NEEDS_HUMAN_REVIEW_LABEL,
@@ -375,6 +382,7 @@ export {
 	NEEDS_HUMAN_REVIEW_LABEL,
 	NOT_DIRECTLY_RUNNABLE_LABELS,
 	OTHER_CHANGE_LABEL,
+	PLANNED_LABEL,
 	PRIORITY_HIGH_LABEL,
 	RELEASE_CLASSIFICATION_LABELS,
 	RELEASE_CLASSIFICATION_NAMES,

package/scripts/issue/issue-lint-cli.ts

@@ -58,7 +58,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv[0])
 }
 
-const issue_lint_cli = { problems, run, OK_MESSAGE, USAGE }
+const issue_lint_cli = { problems, run }
 
 const ARGV_OFFSET = 2
 

package/scripts/issue/issue-merged.ts

@@ -1,5 +1,5 @@
-import { git_gh_api_path } from '#scripts/git/git-gh-api-path'
-import { git_gh_exec } from '#scripts/git/git-gh-exec'
+import { git_gh_api_path } from '#scripts/gh/git-gh-api-path'
+import { git_gh_exec } from '#scripts/gh/git-gh-exec'
 
 // Whether an issue was closed by a merge, read from its REST timeline (joshuafolkken/kit#2701). The
 // issue's own `state_reason` cannot tell: a pull request's `closes #N` and a person's "Close as

package/scripts/{git → issue}/issue-rank.ts

@@ -83,7 +83,7 @@ function rank<T>(items: ReadonlyArray<T>, row_of: (item: T) => RankRow): Array<T
 		.map((entry) => entry.item)
 }
 
-const issue_rank = { compare, count_dependents, is_priority, is_verification_defect, rank }
+const issue_rank = { count_dependents, rank }
 
 export { issue_rank }
 export type { DependencyRow, RankRow }

package/scripts/issue/issue-read-cli.ts

@@ -1,8 +1,9 @@
 #!/usr/bin/env tsx
 import { fileURLToPath } from 'node:url'
-import { git_gh_command } from '#scripts/git/git-gh-command'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { bounded_pool } from '#scripts/lib/bounded-pool'
 import { issue_read, type IssueComment, type IssueFields } from './issue-read'
+import { issue_report_failures, type ReadFailureKind } from './issue-report-failures'
 
 // `josh issue:read <N> [<N> ...]` — the body *and* the comments of every issue named, in one call
 // (joshuafolkken/kit#1715).
@@ -25,7 +26,6 @@ import { issue_read, type IssueComment, type IssueFields } from './issue-read'
 // "no comments" — the block says which of the two it holds, because §2g's rule is that the later text
 // wins, and a comment nobody read cannot win anything.
 
-const SUCCESS_EXIT_CODE = 0
 const FAILURE_EXIT_CODE = 1
 const ARGV_OFFSET = 2
 const ISSUE_NUMBER_PATTERN = /^[1-9]\d*$/u
@@ -35,8 +35,7 @@ const BLOCK_SEPARATOR = '\n\n---\n\n'
 // process, and an unbounded fan-out is answered with secondary rate limiting rather than with issues.
 const READ_CONCURRENCY = 8
 const ISSUE_FIELDS = 'title,state,body'
-
-type ReadFailureKind = 'missing' | 'unreadable'
+const FAILURE_TERMS = { success_kind: 'issue', misreading: 'the issue is empty' } as const
 
 interface IssueContent {
 	fields: IssueFields
@@ -69,16 +68,6 @@ function parse_numbers(argv: ReadonlyArray<string>): ReadonlyArray<string> | und
 	return [...new Set(numbers)]
 }
 
-function report_failure(kind: ReadFailureKind, issue_number: string): void {
-	if (kind === 'missing') {
-		console.error(`✖ issue #${issue_number} does not resolve — check the number and the repository`)
-	} else {
-		console.error(
-			`✖ could not read issue #${issue_number} — a rate limit, expired auth, or a dropped connection. This is not "the issue is empty"`,
-		)
-	}
-}
-
 // The comments are read beside the fields rather than after them: the two requests need nothing from
 // one another, and reading them in sequence would put back one of the two round trips this command
 // exists to remove.
@@ -126,34 +115,6 @@ function issue_blocks(report: IssueReport): ReadonlyArray<string> {
 	return [issue_read.format_issue(report.issue_number, fields, comments)]
 }
 
-function print_issues(reports: ReadonlyArray<IssueReport>): void {
-	const blocks = reports.flatMap((report) => issue_blocks(report))
-
-	if (blocks.length === 0) return
-
-	console.info(blocks.join(BLOCK_SEPARATOR))
-}
-
-function report_one_failure(report: IssueReport): boolean {
-	if (report.result.kind === 'issue') return false
-
-	report_failure(report.result.kind, report.issue_number)
-
-	return true
-}
-
-// Every number that produced no issue is named, so a caller is told which ones it has no answer for
-// rather than being left to subtract the printed blocks from what it asked.
-function report_failures(reports: ReadonlyArray<IssueReport>): number {
-	let has_failure = false
-
-	for (const report of reports) {
-		has_failure = report_one_failure(report) || has_failure
-	}
-
-	return has_failure ? FAILURE_EXIT_CODE : SUCCESS_EXIT_CODE
-}
-
 async function run(argv: ReadonlyArray<string>): Promise<number> {
 	const issue_numbers = parse_numbers(argv)
 
@@ -165,9 +126,12 @@ async function run(argv: ReadonlyArray<string>): Promise<number> {
 
 	const reports = await read_all(issue_numbers)
 
-	print_issues(reports)
+	issue_report_failures.print_blocks(
+		reports.flatMap((report) => issue_blocks(report)),
+		BLOCK_SEPARATOR,
+	)
 
-	return report_failures(reports)
+	return issue_report_failures.report_failures(reports, FAILURE_TERMS)
 }
 
 async function main(argv: ReadonlyArray<string>): Promise<void> {
@@ -175,10 +139,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 }
 
 const issue_read_cli = {
-	BLOCK_SEPARATOR,
-	ISSUE_FIELDS,
 	USAGE,
-	main,
 	parse_numbers,
 	read_block,
 	run,

package/scripts/issue/issue-read.ts

@@ -130,8 +130,6 @@ function format_issue(
 
 const issue_read = {
 	ISSUE_LABEL,
-	TITLE_LABEL,
-	STATE_LABEL,
 	NO_BODY,
 	NO_COMMENTS,
 	COMMENTS_UNREADABLE,

package/scripts/issue/issue-report-failures.ts

@@ -0,0 +1,80 @@
+// The failure report `issue:read` and `issue:state` share (joshuafolkken/kit#3046). Both read a batch
+// of numbers, print a block for each one that answered, and name every one that did not; only the
+// result kind that counts as an answer and what a gap must not be mistaken for differ between them.
+
+const SUCCESS_EXIT_CODE = 0
+const FAILURE_EXIT_CODE = 1
+
+// The two failure kinds a read can end in, kept apart all the way to the report: `missing` is the
+// number resolving to nothing and `unreadable` is a read that failed, and the documents ask a
+// caller to report the two differently. Folding them together to simplify the batch would destroy
+// exactly that distinction.
+type ReadFailureKind = 'missing' | 'unreadable'
+
+interface NumberReport<K extends string> {
+	issue_number: string
+	result: { kind: K | ReadFailureKind }
+}
+
+interface FailureTerms<K extends string> {
+	success_kind: K
+	// What a failed read would be misread as if it were printed silently — "the issue is empty".
+	misreading: string
+}
+
+function print_blocks(blocks: ReadonlyArray<string>, separator: string): void {
+	if (blocks.length === 0) return
+
+	console.info(blocks.join(separator))
+}
+
+// A number that resolves to nothing is an answer — a typo, or another repository's number quoted in
+// prose — and it is reported as one. Everything else is a gap, and the message says so, because the
+// caller's next move differs: a gap is retried, an answer is not.
+function report_failure(kind: ReadFailureKind, issue_number: string, misreading: string): void {
+	if (kind === 'missing') {
+		console.error(`✖ issue #${issue_number} does not resolve — check the number and the repository`)
+	} else {
+		console.error(
+			`✖ could not read issue #${issue_number} — a rate limit, expired auth, or a dropped connection. This is not "${misreading}"`,
+		)
+	}
+}
+
+function is_failure_kind(kind: string): kind is ReadFailureKind {
+	return kind === 'missing' || kind === 'unreadable'
+}
+
+function report_one_failure<K extends string>(
+	report: NumberReport<K>,
+	terms: FailureTerms<K>,
+): boolean {
+	const { kind } = report.result
+
+	if (!is_failure_kind(kind)) return false
+
+	report_failure(kind, report.issue_number, terms.misreading)
+
+	return true
+}
+
+// Every number that produced no answer is named, so a caller is told which ones it has no answer for
+// rather than being left to subtract the printed blocks from what it asked. A single failure makes
+// the exit code non-zero.
+function report_failures<K extends string>(
+	reports: ReadonlyArray<NumberReport<K>>,
+	terms: FailureTerms<K>,
+): number {
+	let has_failure = false
+
+	for (const report of reports) {
+		has_failure = report_one_failure(report, terms) || has_failure
+	}
+
+	return has_failure ? FAILURE_EXIT_CODE : SUCCESS_EXIT_CODE
+}
+
+const issue_report_failures = { print_blocks, report_failures }
+
+export type { ReadFailureKind }
+export { issue_report_failures }

package/scripts/issue/issue-scout-cli.ts

@@ -5,7 +5,7 @@ import { epic_audit_logic } from '#scripts/epic/epic-audit'
 import { epic_bundle, type BacklogIssue, type BundleDecision } from '#scripts/epic/epic-bundle'
 import { epic_bundle_cli } from '#scripts/epic/epic-bundle-cli'
 import { epic_bundle_gaps } from '#scripts/epic/epic-bundle-gaps'
-import { git_gh_command } from '#scripts/git/git-gh-command'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { cli_flags } from '#scripts/lib/cli-flags'
 import { issue_citation } from '#scripts/rules/issue-citation'
 import {
@@ -403,16 +403,10 @@ const issue_scout_cli = {
 	NO_REFERENCE_LINE,
 	CLOSED_UNREADABLE_LINE,
 	CLOSED_CEILING_LINE,
-	DRAFT_NUMBER,
 	read_arguments,
 	with_draft_body,
-	draft_of,
-	format_duplicates,
-	format_epic_decision,
-	format_report,
 	scout,
 	run,
-	main,
 }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))

package/scripts/issue/issue-scout-closed.ts

@@ -1,9 +1,9 @@
-import { git_gh_command } from '#scripts/git/git-gh-command'
-import { EPIC_LABEL, has_any_label } from '#scripts/git/issue-labels'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { PAGE_CEILING_CAUSE } from '#scripts/git/listing-cutoff'
 import { parse_json_array_or_undefined } from '#scripts/git/parse-json-array'
 import { issue_label_schema } from '#scripts/git/schemas'
 import { z } from 'zod'
+import { EPIC_LABEL, has_any_label } from './issue-labels'
 import type { ScoutIssue } from './issue-scout'
 
 // The closed half of `issue:scout`'s duplicate scan (joshuafolkken/kit#1679), held apart from the CLI

package/scripts/issue/issue-scout.ts

@@ -232,10 +232,8 @@ function find_duplicates(
 
 const issue_scout = {
 	SIMILARITY_THRESHOLD,
-	MIN_SHARED_TOKENS,
 	MAX_CANDIDATES,
 	tokenize,
-	shared_tokens,
 	dice_similarity,
 	find_duplicates,
 }

package/scripts/issue/issue-state-cli.ts

@@ -1,7 +1,8 @@
 #!/usr/bin/env tsx
 import { fileURLToPath } from 'node:url'
-import { git_gh_command } from '#scripts/git/git-gh-command'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { bounded_pool } from '#scripts/lib/bounded-pool'
+import { issue_report_failures, type ReadFailureKind } from './issue-report-failures'
 import { issue_state, type IssueState } from './issue-state'
 
 // `josh issue:state <N> [<N> ...]` — print each issue's state and labels, in the spelling the
@@ -25,7 +26,6 @@ import { issue_state, type IssueState } from './issue-state'
 // answer as "not CLOSED" would report a child as failed because nobody could reach GitHub. The two
 // are told apart here and neither is a state.
 
-const SUCCESS_EXIT_CODE = 0
 const FAILURE_EXIT_CODE = 1
 const ARGV_OFFSET = 2
 const REPO_FLAG = '--repo'
@@ -43,18 +43,13 @@ const READ_CONCURRENCY = 8
 // The two fields, under the names `gh issue view --json` gave them. `git-gh-issue-rest.ts` maps them
 // back from REST, which is what keeps `OPEN` / `CLOSED` / `MERGED` out of this file.
 const STATE_FIELDS = 'state,labels'
+const FAILURE_TERMS = { success_kind: 'state', misreading: 'the issue is open' } as const
 
 interface StateRequest {
 	issue_numbers: ReadonlyArray<string>
 	repo?: string
 }
 
-// The two failure kinds a read can end in, kept apart all the way to the report: `missing` is the
-// number resolving to nothing and `unreadable` is a read that failed, and the documents ask a
-// caller to report the two differently. Folding them together to simplify the batch would destroy
-// exactly that distinction.
-type ReadFailureKind = 'missing' | 'unreadable'
-
 // One issue's state, or the failure kind. Named for export because `run:prep` reads the state beside
 // the body and the dependency-update scope (joshuafolkken/kit#1978), reusing this read rather than a
 // second one that would drift from the `human_review` decision `parse_issue_state` owns.
@@ -146,19 +141,6 @@ function parse_request(argv: ReadonlyArray<string>): StateRequest | undefined {
 	return repo_flag.kind === 'named' ? { issue_numbers, repo: repo_flag.repo } : { issue_numbers }
 }
 
-// A number that resolves to nothing is an answer — a typo, or another repository's number quoted in
-// prose — and it is reported as one. Everything else is a gap, and the message says so, because the
-// caller's next move differs: a gap is retried, an answer is not.
-function report_failure(kind: ReadFailureKind, issue_number: string): void {
-	if (kind === 'missing') {
-		console.error(`✖ issue #${issue_number} does not resolve — check the number and the repository`)
-	} else {
-		console.error(
-			`✖ could not read issue #${issue_number} — a rate limit, expired auth, or a dropped connection. This is not "the issue is open"`,
-		)
-	}
-}
-
 // One number's read, reduced to what the report needs and nothing printed yet. Separating the two
 // is what lets the whole batch be in flight at once: the numbers are independent, so reading them
 // one after the other spends a round trip per number for no reason.
@@ -206,30 +188,7 @@ function print_states(reports: ReadonlyArray<IssueReport>): void {
 	const should_attribute = reports.length > 1
 	const blocks = reports.flatMap((report) => state_blocks(report, should_attribute))
 
-	if (blocks.length === 0) return
-
-	console.info(blocks.join(BLOCK_SEPARATOR))
-}
-
-function report_one_failure(report: IssueReport): boolean {
-	if (report.result.kind === 'state') return false
-
-	report_failure(report.result.kind, report.issue_number)
-
-	return true
-}
-
-// Every number that produced no state is named, so a caller is told which ones it has no answer for
-// rather than being left to subtract the printed blocks from what it asked. A single failure still
-// makes the exit code non-zero, exactly as it did when only one number could be passed.
-function report_failures(reports: ReadonlyArray<IssueReport>): number {
-	let has_failure = false
-
-	for (const report of reports) {
-		has_failure = report_one_failure(report) || has_failure
-	}
-
-	return has_failure ? FAILURE_EXIT_CODE : SUCCESS_EXIT_CODE
+	issue_report_failures.print_blocks(blocks, BLOCK_SEPARATOR)
 }
 
 async function run(argv: ReadonlyArray<string>): Promise<number> {
@@ -245,7 +204,7 @@ async function run(argv: ReadonlyArray<string>): Promise<number> {
 
 	print_states(reports)
 
-	return report_failures(reports)
+	return issue_report_failures.report_failures(reports, FAILURE_TERMS)
 }
 
 async function main(argv: ReadonlyArray<string>): Promise<void> {

package/scripts/issue/issue-state.ts

@@ -1,6 +1,6 @@
-import { has_any_label, NEEDS_HUMAN_REVIEW_LABEL } from '#scripts/git/issue-labels'
 import { parse_json_object_safe } from '#scripts/git/parse-json-array'
 import { z } from 'zod'
+import { has_any_label, NEEDS_HUMAN_REVIEW_LABEL } from './issue-labels'
 
 // The answer `josh issue:state` prints, kept apart from the reading and the printing so the shape of
 // the report is decided by one pure function (joshuafolkken/kit#1054).
@@ -96,10 +96,6 @@ function format_attributed_issue_state(issue_number: string, state: IssueState):
 
 const issue_state = {
 	NO_LABELS,
-	STATE_LABEL,
-	LABELS_LABEL,
-	HUMAN_REVIEW_LABEL_LINE,
-	ISSUE_LABEL,
 	parse_issue_state,
 	format_issue_state,
 	format_attributed_issue_state,

package/scripts/issue/markdown-section.ts

@@ -68,6 +68,6 @@ function section_lines(body: string, heading: string): ReadonlyArray<string> {
 	return end === -1 ? rest : rest.slice(0, end)
 }
 
-const markdown_section = { section_lines, is_heading, has_line, has_unfenced_line }
+const markdown_section = { section_lines, has_line, has_unfenced_line }
 
 export { markdown_section }

package/scripts/issue/measure-rerun-cli.ts

@@ -90,7 +90,7 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 	process.exitCode = await run(argv[0], new Date())
 }
 
-const measure_rerun_cli = { run, rerun_one, USAGE, NO_BASELINE }
+const measure_rerun_cli = { run }
 
 const ARGV_OFFSET = 2
 

package/scripts/issue/reproduction-measure.ts

@@ -53,7 +53,7 @@ function has_command_output(body: string, heading: string = REPRODUCTION_HEADING
 	return reproduction.has_command && reproduction.has_output
 }
 
-const reproduction_measure = { REPRODUCTION_HEADING, has_command_output, parse_reproduction }
+const reproduction_measure = { has_command_output }
 
 export type { Reproduction }
 export { reproduction_measure }

package/scripts/josh/cli-body.ts

@@ -91,6 +91,6 @@ function resolve(input: {
 	return read_file_or_stdin(file_path)
 }
 
-const cli_body = { read_file_or_stdin, expand_escaped_newlines, has_value, resolve }
+const cli_body = { read_file_or_stdin, has_value, resolve }
 
 export { cli_body }

package/scripts/josh/file-map-stamp.ts

@@ -253,11 +253,8 @@ const file_map_stamp = {
 	changed_since,
 	create,
 	describes_base,
-	is_file_map,
-	is_writer_gone,
 	is_writer_running,
 	parse_stamp,
-	read_at,
 }
 
 export type { FileMapStamp, FileMapStampAccess }