Repository navigation
feat: consolidate repository quality and diagnostics improvements - #192
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3b213e9245
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 75279457d8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c73d124644
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 48219572a7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b0b12c82b6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c97de4f48d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Problem
The repository presented conflicting component publication status, its distribution policy could miss newly added Marketplace workflows or accept an unrelated retention marker, and relationship diagnostics did not preserve locations already known by the parser. These changes were previously split across #189, #190, and #191, which made their shared compatibility evidence and review fixes harder to assess together.
Solution
This PR supersedes #189, #190, and #191 and includes fixes for every substantive review thread raised on those PRs.
Relevant files
README.md,README.pt-BR.md,SECURITY.md,SUPPORT.md,docs/compatibility*.md, and release documentationscripts/component-distribution-policy-test.shand distribution workflowssrc/AdrGuard/Validation, validation output models/renderers, and JSON/SARIF schemasextensions/vscode/src, its contract fixtures, and integration testsdocs/architecture/component-boundaries*.mdCompatibility
Existing commands, configuration, exit codes, schema versions, and required fields remain unchanged. Diagnostic
lineandcolumnfields are optional and omitted when no reliable source location exists. The extension retries older CLIs that do not support diagnostic locations. Component versions remain independent, and Marketplace publishing stays disabled.Validation
dotnet restore AdrGuard.slnxdotnet build AdrGuard.slnx --configuration Release --no-restore— 0 warnings, 0 errorsdotnet test --solution AdrGuard.slnx --configuration Release --no-build— 695 passednpm ci, lint, typecheck, unit, extension-host, multi-root, restricted-workspace, and README-image checks — 115 unit tests passed, 1 expected platform skip, 17 host tests passednpm ci,npm run sync:docs,npm run validate, andnpm run test:spelling— 116 pages built, 46 bilingual routes, 0 spelling issuesscripts/component-distribution-policy-test.shplus negative mutations for decoy retention and a newly added Marketplace workflowgit diff --checkRisks
The workflow policy intentionally uses conservative text checks and can require an update when GitHub Marketplace publication mechanisms change. Precise locations are emitted only where the parser has evidence; findings without evidence keep the established file-level fallback.