feat(vscode+data): unit tests, perf budgets, input validation, searchability tests - #929
Merged
Gbangbolaoluwagbemiga merged 2 commits intoJun 25, 2026
Conversation
…d searchability tests - vscode-extension: add 27 unit tests covering all analyzer.ts detection paths (auth gaps, panic/unwrap/expect, arithmetic overflow, deduplication, edge cases) using Node's built-in test runner — no new npm deps required - vscode-extension: add performance budget test (500-line contract < 100ms) - vscode-extension: add status bar item showing analysis state and hint count (HyperSafeD#606) - vscode-extension: harden debounceMs clamping to config-declared min/max (HyperSafeD#606) - vscode-extension: validate sanctifierPath on disk when config changes (HyperSafeD#606) - vscode-extension: detect and surface wasm32 compilation failures with actionable error messages and 'Show Error Output' action (HyperSafeD#618) - vscode-extension: add test + lint scripts to package.json so CI passes - data/schemas: add test_searchability.py with 22 tests covering ID indexability, category/severity facets, full-text search fields, regex pattern validity, schema field declarations, and fixture shape validation (HyperSafeD#666) Closes HyperSafeD#606 Closes HyperSafeD#618 Closes HyperSafeD#666 Closes HyperSafeD#740
|
@Gbangbolaoluwagbemiga is attempting to deploy a commit to the gbangbolaoluwagbemiga's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@CelestinaBeing Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
Gbangbolaoluwagbemiga
merged commit Jun 25, 2026
5dcc852
into
HyperSafeD:main
14 of 23 checks passed
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
2 participants
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.
Summary
analyzer.tscovering all detection paths (auth gaps, panic/unwrap/expect, arithmetic overflow, deduplication, clean-contract baseline, edge cases) using Node's built-innode:testrunner — zero new npm dependencies.debounceMsclamping;sanctifierPathpath-existence validation on config change; WASM error detection with actionable "Show Error Output" action in the CLI command.testandlintscripts added topackage.jsonso the VS Code Extension CI job passes end-to-end (npm ci → compile → test → lint → vsce package).test_searchability.pyadds 22 tests across six test classes: ID indexability (unique, non-empty, format, lexicographic sort), category facets (kebab-case, non-empty, coverage), severity facets (valid values, lowercase), full-text search fields (name/description/recommendation non-empty), regex pattern validity, schema field declarations, and canonical fixture shape.CI impact
Issues closed
Closes #606
Closes #618
Closes #666
Closes #740