Origin, app and report refusals live in one module, read by the API and MCP #5690

A new module, src/actions/origin_app_words.ts, now holds the three refusal sentences as ORIGIN_UNREACHABLE, APP_ALREADY_YOURS and REPORT_ALREADY_DECIDED. No module owned these sentences before, so it is a new file. src/http/api.ts and src/http/mcp.ts import them in place of their typed copies, and only the lines that held the sentences changed. In api.ts, the origin-mutation error "the origin could not be reached; nothing was changed" becomes ` ${ORIGIN_UNREACHABLE}; nothing was changed `. It gives the same bytes. No wording changed anywhere.

test/origin-app-words.test.ts checks that the module says each sentence word for word. It also reads api.ts and mcp.ts as text and checks that each imports the module, uses each constant and no longer contains any of the literals, so it fails if a copy is typed back.

Not done: running grep -rn under src/ still finds the sentences outside the module, so that criterion is not fully met. They sit in files the issue does not name, and its risk note keeps the change to api.ts and mcp.ts:

• src/origins/mirror.ts:300 (an exact copy)
• src/actions/migrate.ts:666
• src/tui/screens/common.ts:883 ("…; nothing changed")
• src/contract/openapi.ts:48 and :732 (the contract's descriptions)

Moving them in is a natural follow-up issue.

Checks: bun x tsc --noEmit is clean. The new test passes, and so do the suites covering these refusals: make-it-yours, mcp-origin-apps, mcp-origins, mcp-reports, reports, origins, api-errors, mcp-docs, mcp-migrate and mcp (138 pass, 0 fail). Nothing under src/web changed, so design-lint was not needed.

To try it: run bun test test/origin-app-words.test.ts, then put one of the sentences back as a literal in api.ts and watch the test fail.

🤖 Generated with Claude Code

Closes #5647

A new module, src/actions/origin_app_words.ts, now holds the three refusal sentences as ORIGIN_UNREACHABLE, APP_ALREADY_YOURS and REPORT_ALREADY_DECIDED. No module owned these sentences before, so it is a new file. src/http/api.ts and src/http/mcp.ts import them in place of their typed copies, and only the lines that held the sentences changed. In api.ts, the origin-mutation error "the origin could not be reached; nothing was changed" becomes ` ${ORIGIN_UNREACHABLE}; nothing was changed `. It gives the same bytes. No wording changed anywhere.

test/origin-app-words.test.ts checks that the module says each sentence word for word. It also reads api.ts and mcp.ts as text and checks that each imports the module, uses each constant and no longer contains any of the literals, so it fails if a copy is typed back.

Not done: running grep -rn under src/ still finds the sentences outside the module, so that criterion is not fully met. They sit in files the issue does not name, and its risk note keeps the change to api.ts and mcp.ts:

• src/origins/mirror.ts:300 (an exact copy)
• src/actions/migrate.ts:666
• src/tui/screens/common.ts:883 ("…; nothing changed")
• src/contract/openapi.ts:48 and :732 (the contract's descriptions)

Moving them in is a natural follow-up issue.

Checks: bun x tsc --noEmit is clean. The new test passes, and so do the suites covering these refusals: make-it-yours, mcp-origin-apps, mcp-origins, mcp-reports, reports, origins, api-errors, mcp-docs, mcp-migrate and mcp (138 pass, 0 fail). Nothing under src/web changed, so design-lint was not needed.

To try it: run bun test test/origin-app-words.test.ts, then put one of the sentences back as a literal in api.ts and watch the test fail.

🤖 Generated with Claude Code