-
Notifications
You must be signed in to change notification settings - Fork 0
fix(coverage): resolve npm workspace lock owners #703
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
cc6a54a
fix(coverage): add trusted npm workspace lock resolver
seonghobae fb013f8
test(coverage): cover npm workspace lock resolution
seonghobae dc2fcbf
fix(coverage): bind npm workspace ownership to validated git blobs
seonghobae 840d1d0
test(coverage): prove npm workspace resolver trust boundaries
seonghobae 9f524d6
chore(coverage): bootstrap focused workspace wiring
seonghobae 1f44fd6
chore(ci): retrigger workspace coverage bootstrap
seonghobae a7f34fd
test(coverage): specify npm workspace trust hardening
seonghobae 838346a
ci: exercise npm workspace resolver hardening
seonghobae 18c65da
ci: expose focused resolver tests on pull requests
seonghobae a281b36
chore(coverage): publish bounded bootstrap diagnostics
seonghobae b7b2a54
test(coverage): fix resolver fixture initialization
seonghobae af14b7b
ci: run coverage bootstrap on same-repo PR synchronize
seonghobae b9976fb
fix(coverage): harden npm workspace glob and CLI contracts
seonghobae 99a96bd
test(coverage): close resolver branch coverage gaps
seonghobae ab50cc3
chore(coverage): trigger one-shot workspace wiring
seonghobae 2623938
fix(coverage): trust validated HEAD workspace ownership
seonghobae 89a6ff0
test(coverage): accept bounded HEAD workspace updates
seonghobae ab51380
test(coverage): harden anchored workspace matching
seonghobae fbc0a91
fix(coverage): scope offline npm installs to selected workspace
seonghobae f1ee192
ci(coverage): verify workspace wiring before commit
seonghobae 05d3b84
fix(coverage): honor Git clean filters when validating npm workspace …
seonghobae 96dade5
test(coverage): pin path-aware npm worktree hashing
seonghobae 72f4088
fix(coverage): preserve Git path filters in generated npm lock verifi…
seonghobae d573869
fix(ci): pass bootstrap result through the step environment
seonghobae ad8a9ae
chore(coverage): trigger verified workspace wiring bootstrap
seonghobae 558d448
chore(ci): retrigger verified npm workspace bootstrap
seonghobae d0323b1
chore(ci): retrigger verified npm workspace wiring
seonghobae b7f12da
chore(ci): add bounded PR 703 final repair
seonghobae 69e5fcc
chore(ci): trigger bounded PR 703 final repair
seonghobae a70578a
ci(pr703): generate reviewed repair artifact without write credentials
seonghobae d3abaa7
ci(pr703): run final repair validation for the pull request
seonghobae 8e5b5fd
fix(pr703): emit valid structured invocation contract
seonghobae 20c3ce6
fix(pr703): parse the complete nested npm case contract
seonghobae 04a23b7
fix(pr703): keep generated nested-case delimiter syntactically valid
seonghobae 309ed34
fix(pr703): assert every npm failure path in the complete case
seonghobae accbf69
fix(pr703): assert all bounded npm returns
seonghobae 964ea61
fix(pr703): publish verified canonical final state
seonghobae 2661025
ci(pr703): preserve verified canonical final state
seonghobae e292fa8
fix(coverage): finalize npm workspace lock resolution
seonghobae cf65dee
ci(deps): refresh vulnerable Strix dependency lock
seonghobae 6ad0ffe
fix(deps): update Strix cryptography and aiohttp
github-actions[bot] 8af1d77
chore(ci): remove temporary Strix lock refresher
seonghobae f6acae6
fix(coverage): prepare resolver review cleanup
seonghobae 40d794a
ci(coverage): verify resolver review cleanup
seonghobae 6b43ac5
refactor(coverage): address npm resolver review feedback
seonghobae 6e29a57
test(coverage): repair resolver hardening regression
seonghobae e9a4583
test(coverage): restore complete resolver contract suite
seonghobae bb129fe
fix(security): align scheduled CodeQL action version
seonghobae e23cba9
chore(coverage): remove unrelated scheduled security drift
seonghobae b824259
fix(ci): align CodeQL action refs for coverage
seonghobae c26f40c
ci(repair): verify PR 703 review feedback
seonghobae File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
명령 치환으로 호출되는 두 해석 함수가 진단을 stdout으로 출력합니다.
install_package_dependencies는 두 함수의 stdout을 결과 값으로 캡처합니다. 따라서::error::메시지는 로그에 남지 않고 버려집니다..github/workflows/opencode-review-dispatch.yml#L1249-L1268:resolve_npm_package_root의echo "::error::..."4곳에>&2를 추가하십시오..github/workflows/opencode-review-dispatch.yml#L1286-L1312:resolve_npm_install_root의echo "::error::..."4곳에>&2를 추가하십시오.📍 Affects 1 file
.github/workflows/opencode-review-dispatch.yml#L1249-L1268(this comment).github/workflows/opencode-review-dispatch.yml#L1286-L1312🤖 Prompt for AI Agents