-
Notifications
You must be signed in to change notification settings - Fork 0
feat(licensing): generate deterministic dependency license inventory (#495 merge-result promotion) #506
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
feat(licensing): generate deterministic dependency license inventory (#495 merge-result promotion) #506
Changes from all commits
6ca44bb
db11628
db9262f
be8c078
8a1ab14
0712eb2
c68fd9d
c54a49f
21d5033
6cc2609
25165a8
efca994
58cef0d
8e49b1b
4b0adec
b9b146c
e35eaab
f24e63b
f2b60e2
4ad0554
5445071
0f8086a
95e4ca2
ba093ec
8fc000c
f6a6fb7
2f62973
f5c3fe1
8482729
b5aaee6
d22b927
9223658
9b361fe
0359e56
2e98cdf
4829226
643b8bd
fd17025
b4f34d8
7f8ea91
b07aa86
66d673e
4925e0c
6d5a6f8
90820b7
af782ab
1678a26
00d4b29
f650505
5f9105a
7432823
8dc6028
c7be943
d7977cf
6d535cf
28c881c
6ab9ca9
8883ce2
13420ce
b5cd9d4
a07d654
65e1d76
291cb31
8954d2c
edee977
9014834
3bf3821
5c8489b
d3285d3
65d4b5e
6b6d7e3
5166412
293cdb4
fee5408
2bc835a
066b48f
c6a9784
d1f70bc
de730b7
6eee6c5
04c5658
c1297d1
3b5ca1d
a340d4d
4a2ba84
c13b96b
c6baed6
5c823ad
f28bd0c
0ce5437
eec955c
fba3343
b5b3930
f8a2d7b
2b9ca10
9f88f5f
9e92dcf
ef7db03
2f2afb3
9120b26
a104d51
63e30bc
6bd4305
5eeda68
f40420c
c57294b
7b2b896
598b8f3
80e56af
b3aa8f3
3621bac
b5c4eba
618edf7
3886de9
9dc2c19
0ec3e10
6bb429f
3023285
4ed493b
4f26bc6
28d7699
85c0021
2ab5581
f9bd0bf
60d6d7e
666d33f
85b9f8d
3b0d1b8
bc7d80f
07073b0
3925296
1507ce2
a7dcc70
7c51620
b23b170
4e65d6f
52333fb
7cb56cd
b62e872
d6c76ee
1341098
5dbb8d7
21c0197
475f49f
454d0ab
6e2d90a
9121b00
2cc9b46
0c8725b
79ddf35
2ec94fb
8305187
eb7f35a
20f7174
9e38a84
064d565
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -36,9 +36,10 @@ | |||||
| "smoke:check": "bash scripts/smoke-readiness.sh", | ||||||
| "release:evidence": "node scripts/release-evidence.mjs", | ||||||
| "release:publication-receipt": "node scripts/release-publication-receipt.mjs", | ||||||
| "release:dependency-license-inventory": "node scripts/dependency-license-inventory.mjs", | ||||||
| "deployment:evidence": "node scripts/deployment-evidence.mjs", | ||||||
| "release:verify": "npm run typecheck && npm run test && npm run security:scan && npm run kpi:verify && npm run acquisition:manifest && npm run acquisition:integrity", | ||||||
| "release:verify:strict": "npm run typecheck && npm run test && npm run security:scan && npm run kpi:verify:strict && npm run acquisition:manifest && npm run acquisition:integrity", | ||||||
| "release:verify": "npm run typecheck && npm run test && npm run security:scan && npm run kpi:verify && npm run release:dependency-license-inventory && npm run acquisition:manifest && npm run acquisition:integrity", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟡 Required CI skips license inventory The required CI workflow bypasses Prompt for agentsWas this helpful? React with 👍 or 👎 to provide feedback. |
||||||
| "release:verify:strict": "npm run typecheck && npm run test && npm run security:scan && npm run kpi:verify:strict && npm run release:dependency-license-inventory && npm run acquisition:manifest && npm run acquisition:integrity", | ||||||
|
Comment on lines
+39
to
+42
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||
| "security:scan": "npm audit --audit-level=high", | ||||||
| "security:evidence": "node scripts/security-validation-evidence.mjs", | ||||||
| "security:private-reporting-audit": "node scripts/private-vulnerability-reporting-audit.mjs", | ||||||
|
|
@@ -54,7 +55,7 @@ | |||||
| "patch-validator:image:verify-receipts": "node scripts/verify-patch-validator-image.mjs", | ||||||
| "acquisition:deployment-evidence": "node scripts/acquisition-deployment-evidence-audit.mjs", | ||||||
| "acquisition:integrity": "node scripts/acquisition-data-room-integrity-audit.mjs", | ||||||
| "acquisition:audit": "npm run acquisition:integrity && node scripts/acquisition-readiness-audit.mjs && npm run acquisition:deployment-evidence", | ||||||
| "acquisition:audit": "npm run release:dependency-license-inventory && npm run acquisition:integrity && node scripts/acquisition-readiness-audit.mjs && npm run acquisition:deployment-evidence", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟡 Scheduled acquisition audits always fail After a fresh manifest records no inventory,
Suggested change
Was this helpful? React with 👍 or 👎 to provide feedback. |
||||||
| "acquisition:manifest": "node scripts/acquisition-data-room-manifest.mjs", | ||||||
| "readiness:audit": "node scripts/saleable-readiness-audit.mjs" | ||||||
| }, | ||||||
|
|
||||||
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
승격 후 PR 상태와 protected
main식별자를 갱신하세요.PR 목표에 따르면 이 변경은 PR
#495의병합 결과를main으로 승격하고, PR#495는검증 후 superseded로 종료됩니다. 현재 문구는 병합 후에도 PR#495가활성 상태이고 생성기가 protected-main 이전의 증거라고 잘못 설명합니다.Line 150의 SHA도 승격 대상 base 또는 promotion head와 일치하지 않습니다. 인수 증거 기준선에는 병합된 protectedmain의 실제 식별자와 상태를 기록하세요.docs/LICENSING_AND_IP_TRANSFER.md#L3-L3: 활성 PR#495상태를 병합된 증거 상태로 변경하세요.docs/LICENSING_AND_IP_TRANSFER.md#L88-L89: 생성기를 진행 중인 PR 변경이 아닌 병합된 릴리스 증거 생성기로 설명하세요.docs/LICENSING_AND_IP_TRANSFER.md#L150-L150: protectedmainSHA를 이 승격의 실제 병합 commit으로 갱신하세요.docs/LICENSING_AND_IP_TRANSFER.md#L158-L158:active-PR truth및Until it integrates문구를 제거하거나 병합 후 상태에 맞게 변경하세요.📍 Affects 1 file
docs/LICENSING_AND_IP_TRANSFER.md#L3-L3(this comment)docs/LICENSING_AND_IP_TRANSFER.md#L88-L89docs/LICENSING_AND_IP_TRANSFER.md#L150-L150docs/LICENSING_AND_IP_TRANSFER.md#L158-L158🤖 Prompt for AI Agents