UZ-WH-020 states both causes, and the fix names a real field - #175
Merged
Conversation
The row said the workspace lacks the named secret. That is one of two causes. The trigger's source can also be unknown to the provider registry, and the middleware answers UZ-WH-020 for both. The fix column pointed at secret_ref, a field from the Svix path. A manual webhook trigger resolves credential_name, which defaults to the source name. The row now names the field the reader's TRIGGER.md actually has. error-codes.mdx is regenerated from the registry, whose UZ-WH-020 entry now states the fact before the fix like every other row. The verified stamp moves forward with the regeneration. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Greptile, twice and correctly. UZ-WH-020 has two causes and the remedy answered only one: creating a secret does nothing when the trigger's source is not a recognized provider, so the sender resends and gets UZ-WH-020 again. Both rows now name the recognized sources — github, slack, linear (webhook_verify.zig) — before the secret step. The webhooks row also never said where the signing value goes. It is the secret's webhook_secret field (serve_webhook_lookup.zig reads exactly that field); storing it under another property leaves the verifier unable to find it. error-codes.mdx is regenerated from the corrected registry entry, which ships in agentsfleet PR #603. The registry string is split into three sentences so the generated row clears this repository's 25-word DOC-02 check, which caught it at 26. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Found while auditing the
agentsfleetarchitecture docs against this repo (M165). The same error code is documented in three places and had drifted in two of them.What was wrong
fleets/webhooks.mdxnamed one cause of two. The row read "The workspace lacks the named secret." The middleware answersUZ-WH-020for two distinct situations —webhook_sig.zig's own module doc lists them:sourceis not a recognized provider, orA user whose
sourceis simply unknown to the registry was pointed at creating a secret they may already have.The fix column named a field that page's reader does not have. It said to create the
secret_refname.secret_refbelongs to the Svix path (serve_webhook_lookup.zigreadssignature.secret_reffrom Clerk-style config). A manual fleet webhook resolvescredential_name, which defaults to thesourcename — andcredential_nameis what the page's ownTRIGGER.mdexample already uses.api-reference/error-codes.mdxcarried a remedy in its Description column. Every other row states the fact; this one opened with "Create a random webhook secret of at least 32 bytes…".What changed
The registry entry in
agentsfleet(src/agentsfleetd/errors/error_entries.zig) now states the fact before the fix and names both causes.error-codes.mdxhere is regenerated from it withmake gen-error-codes— no hand edits, and the generator output is byte-identical to what is committed.fleets/webhooks.mdx's row is rewritten by hand to match, and both pages'verified:stamps move forward.Pairing
The registry change ships in agentsfleet PR #603. Merge that first, or together — the regenerated page here matches its registry byte for byte, so landing this alone would leave the generator output ahead of the code it generates from.
Verification
make lint— passed (22 tests, link check clean)make gen-error-codesoutput diffed against the committed page — identical🤖 Generated with Claude Code
Greptile Summary
The PR corrects the documentation for
UZ-WH-020so both unsupported providers and missing secrets are covered.credential_namelookup behavior and the requiredwebhook_secretfield.Confidence Score: 5/5
The PR appears safe to merge.
No blocking failure remains; both previously reported documentation issues are resolved by the current text.
Important Files Changed
Reviews (2): Last reviewed commit: "docs: a secret cannot make an unsupporte..." | Re-trigger Greptile
Context used: