Add OpenAI Agents JS lockfile example and verified case study#501
Add OpenAI Agents JS lockfile example and verified case study#501Ayush7614 wants to merge 1 commit into
Conversation
Ayush7614
force-pushed
the
ayush12
branch
2 times, most recently
from
eade58d to
45a8b39
Compare
sonukapoor
left a comment
sonukapoor
left a comment
There was a problem hiding this comment.
The all-transitive framing is documented well, and the one concrete verdaccio parent upgrade makes this more useful than a pure zero-output study. The framing note in Summary sets honest expectations.
Same CHANGELOG issue as #500: the second bullet references VS Code in [Unreleased], but VS Code shipped in v1.18.1. Please remove the second bullet or trim it to reference OpenAI Agents JS only.
sonukapoor
left a comment
sonukapoor
left a comment
There was a problem hiding this comment.
The all-transitive framing is well handled — making the case for transitive parent tracing on a high-visibility AI SDK repo is a legitimate angle. Two things before merge:
Please rebase against main. The branch was cut before v1.18.2 shipped, so the CHANGELOG has a conflict. After rebasing, drop the VS Code line from your [Unreleased] CHANGELOG entry — that's already captured in v1.18.2. Keep just the OpenAI Agents JS line.
Add a row to website/docs/case-studies/index.md. A case studies index page was added after your branch was cut — it's on main now. After rebasing you'll find it at website/docs/case-studies/index.md. Suggested row:
| [OpenAI Agents SDK (JS)](./openai-agents-js.md) | pnpm | AI agent monorepo — 1,683 packages, 0 direct findings, 31 transitive, one verdaccio parent-upgrade command |
Everything else looks good — README updated, sidebar entry present, logo committed locally.
Snapshot openai/openai-agents-js pnpm lockfile (1,683 packages, 31 all-transitive findings) with parent-tracing narrative, one verdaccio fix command, pnpm audit comparison, and docs wiring (README, CHANGELOG, case studies index).
|
Thanks for the review @sonukapoor — addressed all requested updates:
Ready for another look. |
2 participants
Summary
examples/openai-agents-js/from openai/openai-agents-js@f76fc19 (package.json+pnpm-lock.yaml).website/docs/case-studies/openai-agents-js.md(1,683 packages, 31 findings, pnpm audit comparison).website/static/img/openai-agents-js-logo.svg.pnpm add verdaccio@6.7.2command, MCP/Daytona clusters.Closes #490
Test plan
npm run build && node dist/index.js examples/openai-agents-js --verbose --all— 31 findings (0 critical · 13 high · 16 medium · 2 low), 0 directpnpm add verdaccio@6.7.2(1/31 first-pass coverage)pnpm audit(Node 22+, pnpm 10.14.0) — 52 entries documentedcd website && npm run build— Docusaurus build succeedsMade with Cursor