docs: port ttl-editor's Semgrep posture into this copy, without losing ours - #90
Merged
Merged
Conversation
…g ours
The CI posture page lives in both repositories and had diverged in both
directions. ttl-editor's copy gained the Semgrep gate -- a fourth mechanism,
the npm dependency tree that check-supply-chain was never able to see -- and
its decision to leave main ungated. This copy gained the Linked Data Explorer
promotion: the main promotion gate, the protected-environment table, the
paths: self-trigger, "behind is not the same as diverged", and checklist items
10 to 12.
Overwriting one with the other would have repeated exactly the mistake 11f7a78
in ttl-editor records -- patching a stale copy. So this ports only the hunks
that exist in ttl-editor's copy and not here:
- the intro paragraph naming the fourth mechanism
- the Semgrep Code + SCA row in the summary table
- the whole "The other supply chain: the npm tree" subsection under
section 2, verbatim -- byte-identical to ttl-editor's, checked
- section 4's account of ttl-editor's ruleset now requiring audit and scan,
and its main asymmetry being decided rather than merely defensible
- open-work rows for ttl-editor #128 and #131
and keeps every section unique to this copy. Each of the five LDE-only
headings was checked present after the port.
One sentence is re-scoped rather than copied. ttl-editor's copy says "every
control this document describes now gates acc and none of them gates main".
True there, and false here, where the section immediately after describes
Linked Data Explorer's own main promotion gate. It now reads as a statement
about ttl-editor, with a pointer to the opposite case below it.
ttl-editor's row in the verified-heads table moves to 7f95502, re-verified;
this repository's own row is left as its last pass recorded it rather than
updated without re-verifying.
A new open-work row records this repository's own gap: no Semgrep scan yet,
so its npm tree is remediated by Renovate and verified by nobody. A local
dry run on cfa6b40 found 86 findings, 66 of them Supply Chain, all transitive
and unreachable. Closing that gap is the next change.
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.
The CI posture page lives in both repositories and had diverged in both directions. This ports ttl-editor's Semgrep work in without overwriting what only this copy has.
Ported from ttl-editor
check-supply-chainwas never able to seeSemgrep Code + SCArow in the summary tableauditandscan, and itsmainasymmetry recorded as decided, not merely defensibleKept, because only this copy has it
main promotion gatesection and the ruleset lessonspaths:filter is a trigger"1.9.12rowAfter the port, each of the five LDE-only headings was checked for presence. Overwriting either copy with the other would have repeated what ttl-editor's
11f7a78records: patching a stale copy.One sentence re-scoped rather than copied
ttl-editor's copy says "every control this document describes now gates
accand none of them gatesmain". That is true there. It is false here, where the next section describes Linked Data Explorer's ownmain promotion gate. It now reads as a statement about ttl-editor, with a pointer to the opposite case.Heads table
ttl-editor's row moves to
7f95502, re-verified. This repository's row stays as its last pass recorded it; I did not update it without re-verifying.This repository's own gap, now recorded
A new open-work row says what is missing here: no Semgrep scan, so the npm tree is remediated by Renovate and verified by nobody.
A local dry run on
cfa6b40found 86 findings:examples/Closing that gap is the next change.
Note for the iou-architectuur patch
This copy is now the more complete of the two. Use it as the source.
Verification
npm run check-formatclean. Documentation only.