docs: add Strapi case study (#373)#593
Conversation
|
cc: @sonukapoor |
|
This branch has merge conflicts with main — could you rebase against main, resolve the conflicts, and force-push? Thanks! |
|
@sonukapoor Rebased onto |
|
@Ayush7614 Could you please resolve the conflicts and push again? |
|
@sonukapoor Rebased onto latest |
|
Rebased to include the |
sonukapoor
left a comment
There was a problem hiding this comment.
The case study content looks solid - the CMS comparison angle with Ghost and the direct vs transitive split are well explained. Three things to fix before we can merge:
- Please revert the
package.jsonchange (thetsxversion bump from4.19.2to4.22.0). Dependency updates are separate from case study PRs. - Please revert the
package-lock.jsonchange - this was generated from the tsx bump above. - Please revert the
CHANGELOG.mdchange - we manage the changelog at release time, not in individual PRs.
Once those three files are reverted to match main, this is ready to merge.
|
@sonukapoor Thanks for the review — addressed all three items:
PR now only touches the Strapi case study files. Ready for another look when you have a moment. |
|
Thanks for the case study contribution. A note for this PR and future ones: please only touch files specific to your study — the case study document, fixture files, and logo. Do not modify these shared files:
When multiple case study PRs are open at the same time, all of them touch these files and conflicts pile up. Maintainers add the index/sidebar/README entries when merging. We have updated CONTRIBUTING.md to document this. For this PR specifically: please rebase against main and resolve any conflicts, keeping the main branch version of the shared files. |
Document strapi/strapi at e666ee2 (2,887 packages, 29 findings) with CMS parallel to Ghost on html-minifier, direct lodash/qs fixes, and six fix groups. Closes OWASP#373
|
@sonukapoor Rebased onto latest PR now only touches Strapi-specific files:
No changes to |
sonukapoor
left a comment
There was a problem hiding this comment.
Clean - only touches Strapi-specific files. Fixture works correctly (2,887 packages). Approved.
|
Merged - thank you @Ayush7614! |
Summary
examples/strapi/lockfile snapshot (Yarn Berry 4.12.0) and documents six fix command groups covering 12/29 findingshtml-minifierno-fix), directlodash/qsfixes vs transitive criticalhandlebarschain, and mixedminimatchremediation pathsyarn npm auditlimitation on lockfile-only Yarn Berry catalog snapshots (same class as Storybook)Closes #373
Verified scan output
Key generated commands:
Test plan
npm run buildnode dist/index.js examples/strapi --verbose --all— 29 findings, 6 command groups, 12/29 coveragecve-lite-scan-2026-06-09T07-16-34.json)yarn npm auditattempted on fixture — fails with catalog protocol error (documented in case study)