Repository navigation
feat(core): add guarded approvals with verified workspace dependencies - #11
Merged
Merged
Conversation
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.
Adds additive
@lingva/coreguarded approval types, named actions, deterministic request builders and sanitized response parsers. Callers provide explicit scope, context token and operation UUID; the codecs preserve identities, validate serialized UTF-8 limits and never infer authorization. Existing authoring contracts and all Lingva package versions/licenses remain unchanged.The PR also repairs the existing workspace dependency-audit blocker so the new contracts can be verified against a maintained dependency graph:
nuxt typecheckbefore production bundling. The duplicate Vite checker shell-splits absolute project paths containing spaces; its optional development overlay is disabled.Validation completed locally:
Production audit exits 0 with no active advisory objects under the explicitly documented exception policy. It still reports two ignored patched-artifact findings: the existing node-forge exception and the new exact deepmerge GHSA-gcqx-w48m-w2h4 exception. The existing braces exception remains unchanged. This is not an exception-free audit claim. See
docs/specs/workspace-dependency-security.mdfor patch provenance, removal conditions, compatibility tradeoffs and rollback.HTTP helpers, CLI/adapters, server/storage/permission behavior, package publication and deployment remain outside this PR. At head
e962e60, CI complete, workspace/docs/native checks and all 22 Snyk security checks pass. Entire reports only missing reviewer approval; Checks, Findings and Up to date pass. No approval bypass or merge has been performed.