Repository navigation
ADR-052: scan defaults for Apps Script, .NET output, Razor and credential files (B-015, B-063, B-064, B-018) - #90
Merged
ethanblauw-hub merged 9 commits intoOct 20, 2026
Conversation
…edential files Co-Authored-By: Claude <noreply@anthropic.com>
…s and Razor scan_policy gains .gs, .cshtml and .razor as default extensions; obj at any depth and MSBuild's bin/Debug, bin/Release, bin/x64, bin/x86 as ignored directories, via multi-segment ignore_dirs entries matched as consecutive directory names; and a files / extra_files knob of file-name patterns that are never scanned, defaulting to common credential files, with ! patterns re-including (.env.example). Co-Authored-By: Claude <noreply@anthropic.com> [ADR-052 src/CLAUDE.md src/scan_policy.py]
…2 defaults AppsScriptAdapter maps .gs to the JS grammar: a bare REGISTRY line raised KeyError because the web adapter picks its grammar by extension. Adds a two-file conformance fixture (1.0/1.0, new apps_script baseline row), the .gs extension in conformance_eval, and tests for cross-file resolution, the obj/bin rules, Razor through the fallback chunker end to end, and the credential-file patterns. Co-Authored-By: Claude <noreply@anthropic.com> [ADR-052 src/adapters/__init__.py]
Co-Authored-By: Claude <noreply@anthropic.com> [ADR-052]
Co-Authored-By: Claude <noreply@anthropic.com> [ADR-052]
…er's own copy Every .gs file shares one global scope, and the copy that loads last wins everywhere, so a bare call with several candidates skips the same-file and import-scoped steps. Member calls and .js callers are unchanged. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> [ADR-052 src/call_resolver.py]
…xtra_extensions Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> [ADR-052 src/CLAUDE.md src/scan_policy.py]
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> [ADR-052]
…n-defaults-and-languages [ADR-052 src/hybrid_retriever.py src/iterative_retriever.py]
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.
Implements ADR-052 (backlog B-015, B-063 item 1, B-064 level 1, B-018 scan half).
What changes
.gsis parsed with the JavaScript grammar (B-015)..gsfile calls a function that two files define, the call stays unresolved. Apps Script has one global scope, so no file is "nearer"..jscallers keep the same-file step.obj/is excluded everywhere;bin/only under MSBuild's configuration folders..cshtmland.razorare indexed as text (B-064 level 1)..xamlstays opt-in..env*, key files and gcloud's*-<12 hex>.json.!nameinextra_dirs,extra_root_dirsorextra_extensionsdrops that one default (§5, @EDB).!that matches no default raises an error, and so does a!in a bare key.extra_fileskeeps its existing re-include meaning.Tests
tests/test_scan_defaults_adr052.py, which includes the Apps Script duplicate rule and six!tests.🤖 Generated with Claude Code