Post-GA follow-up: green the lint/policy gates + PATCH docs + TRACKER (after #194)#195
Merged
Merged
Conversation
Add the PATCH /v1/corpora/{id} and PATCH /v1/webhooks/subscriptions/{id}
update routes to the gateway + webhooks references (summary tables + a corpus
PATCH section with curl example): partial-update semantics, immutable embedding
fields / masked signing secret, 200 + 404 invisibility rule.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- exclude scripts/demo (GTM slide builders + adversarial safety demos) from ruff and from the PolicyEngine coverage linter — demo tooling, not shipped code; safety_demo.py deliberately bulk_indexes a poisoned chunk to show the injection guard catching it, so it has no PolicyEngine call site by design - apply ruff UP017 (datetime.UTC) + ruff format to the shipped corpus/webhook update code (pg_corpus_store, noop stores, pgvector, test_corpora) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Flip the Step 3.10 'update (PATCH) deferred' note to done, add a post-GA batch entry to Recently shipped, refresh the Status block + Last updated. Co-Authored-By: Claude Opus 4.8 (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
1 participant
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.
Why this PR
#194 was merged at its 6-commit state, before three follow-up commits landed. Those commits make the quality gates pass and add the docs/TRACKER updates, so this PR brings them into
main.main's gates are currently red without this PR — at the merged state,ruff checkfails (UP017 + formatting in the shipped corpus/webhook code) andpytestfails (the PolicyEngine coverage linter flagsscripts/demo/safety_demo.py, which deliberately bypasses the policy engine to demo the injection guard). This PR is the fix.What's in it
6f384e0gateway.md/webhooks.md2665a53scripts/demo(GTM tooling) from ruff + the policy-coverage linter;ruff format+UP017on the shipped codef16831eVerification — all gates green (local; CI billing-blocked)
ruff check+ruff format --checkmypy --strict(packages/+apps/gateway/)pytest(full,-m 'not perf')tsc·eslint·vitest·next build🤖 Generated with Claude Code