AI: a provider's refusal in words, the key tested before AI on, and no brand words

[futurebuildai]

Item 130, N3 to N8: the provider's refusal in words, the key tested before AI reads as on, and no brand words.

What changes

• N3, N4 (a refusal is not "no reader" and not a 500). A 401 or 403 from the provider is a refusal of the saved key. The material intake and PIM's descriptions, SEO and collateral routes answer 502 in the wire envelope with the code ai_refused and "Anthropic refused the saved key. An administrator can enter it again in Tech Admin, AI Settings." The no key case is unchanged. A pasted or text list is still read by the rule reader when the key is refused. The intake and the three PIM tabs show the words where they happened. The image route is unchanged: it answers 202 and generates after, so a refusal there is the media row's failed status.
• N7. Saving AI settings runs the provider test and stores its result; the document gains test_status (none, untested, passed, refused, failed) and test_message, and the test route stores its result the same way. Tech Admin reads "AI on" only after a passed test, "AI off: Anthropic refused the saved key" after a refusal, and says a saved key was not tested yet. The characterization harness points the server's provider at a local stand-in, so no test reaches the network.
• N8. The illustration prompt asks for no words, labels, brand names or logos and no longer lists the text elements.
• N5, N6. The intake's sample list and its test fixtures, and the demo import's CN_A35 ("Framing Angle A35"), carry plain product words; the seed's brand check lists "quikrete", "tyvek" and "proframe angle".

Red first (each commit alone, failing on the base)

Leg	Command	At its first commit
Intake refusal	go test ./internal/parsing -run TestWire_UploadProviderRefusal	status 200 with the sample list, want 502
PIM refusal	go test ./internal/pim -run TestWire_PIMGenerateRoutesAnswer	status 500, want 502 (three routes)
Text list under refusal	go test ./internal/parsing -run TestExtractItemsWithAI_TextList	synthetic true
Save tests the key	go test ./internal/techadmin -run 'TestWire_SavingAKey|TestWire_AnUntested'	provider called 0 times, no test_status
Illustration prompt	go test ./internal/pim -run TestIllustrationPrompt	three failures
Brand words	go test ./internal/parsing -run TestSampleList, go test ./internal/app/seed -run TestSeedDataHoldsNoBrandWord	sample list and catalog hold the words
Intake words	vitest MaterialIntake.test.ts	no intake-ai-refused
PIM page	vitest ProductContentTab.test.ts	no pim-ai-notice (three tabs)
Tech Admin statuses	vitest aiWords.test.ts AIProviderWords.test.ts	refused and untested lines absent

All green at the head.

Contract

502 ai_refused on the intake and three PIM routes; test_status, test_message on the AI settings document; status on the test answer. merge -check, cmd/pending, api-client generate, drift and typecheck are clean; CONTRACT-CHANGES has the row; goldens techadmin_settings and machine_key re-recorded for test_status only.

Review round 1

• P2-1 freight: the freight route answers 502 ai_refused with the sentence and "Until then, enter the freight total by hand."; the uploaded file is kept only after a successful read. Leg: go test ./internal/purchase_order -run TestWire_FreightUploadAnswers (400, want 502 at its first commit).
• P2-2 image: a refusal settles the media row refused; the PIM page shows the sentence naming Tech Admin, AI Settings; the failed words no longer say try again. Legs: go test ./internal/pim -run TestWire_PIMImageSettlesRefused, vitest ProductMediaTab.test.ts, aiWords.test.ts.
• P2-3 CI network: the e2e save types http://127.0.0.1:9/api/v1; a guard (e2eNoNetwork.test.ts) fails on any non loopback host in a spec.
• P3s: only a 401 refuses OpenRouter's key; the stored test names its settings revision (a mismatch reads untested); an unreachable provider answers in a sentence, the save's test has its own timeout and a storage error after the commit is logged; the sanitiser drops text and tspan; the extraction prompt example and the tax test are plain and openrouter.go joins the brand check; the intake leg drives the file input; the AI feature grid follows aiIsOn.
• ProFrame: the 74 product descriptions that named it take plain names; vendor records and keys, and the model designations, stay (the principal's call). The seed check bans "proframe" in descriptions.
• Not done: an environment key changed across a restart still keeps an old passed (the revision does not move); PR 147 is not on refactor/v1, so its colour check is not on this branch (the amber notices wait for it); the order of the merge of refactor/v1 is current as of this push.

🤖 Generated with Claude Code