Repository navigation
Catch spaced +91 numbers and update the docs for 0.4.0 - #11
Merged
Merged
Conversation
The guard let +91 98765 43210, the usual written form, and lowercase PAN through, while the README said it blocked +91 numbers. Both block now, with tests, and a landline still passes. The README and SECURITY.md still said no run had measured sector detection after tamarind-lend did. The accuracy table now covers both blind runs with tamarind-lend's caveats, Quick start names Claude Code 2.1.295 as the minimum, and the plugin section stops claiming every skill works in any project. Versions move to 0.4.0, and the Unreleased section becomes its entry. Co-Authored-By: Claude Opus 5.5 <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.
Follow-ups from an independent check of
masterateb1fbd5, after PRs #8, #9 and #10 merged.Guard
+91 98765 43210and+91-98765-43210passed the personal-data filter, though the README says it blocks +91 numbers. Lowercase PAN passed too. All three block now. Four new asserts inguardrails.test.mjs, including a Mumbai landline that must still pass.Docs
onFailure: "block"is ignored, and a guard that fails to start lets the write through./vigil:auditand/vigil:onboardwriteassessments/andcompany-profile/into whatever project is open, unguarded, and/vigil:exportneeds the workspace's npm scripts.AGENTS.md,CLAUDE.mdand.github/.Release
package.json,plugin.jsonandCITATION.cffmove to 0.4.0, and the Unreleased section becomes the 0.4.0 entry. Tagv0.4.0after merge.Checks
npm test: 26 pass, 0 fail, 1 skipped on Windows (CI runs it on Linux).npm run validate -- --strict: 0 errors.Not tested:
claude plugin marketplace addandclaude plugin install. The Claude CLI wasn't available, so run them once before announcing the plugin.🤖 Generated with Claude Code