Repository navigation
docs: CLAUDE.md (API contract, review rules, DoD) and CONTRIBUTING.md - #70
pritesh-jain-cstk wants to merge 4 commits into
Conversation
MKT-22144. Also ignore .env files (the library reads none). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
MKT-22144. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
…text MKT-22144. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
Drop the internal workflow section and internal tracker keys; link the host-only listener fix by PR instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
Branch graph
Epic MKT-21682. This PR is highlighted. Only [Routine] depends on [Tests]; the rest are independent.
Merge order: #68 before #69 (#69 is based on #68). #64 and #69 both change
package.jsonandpackage-lock.json: merge #64 first, then bringmasterinto #69 with a merge (no rebase) and regenerate the lock withnpm install --legacy-peer-deps(expect a lock conflict). #65, #66, #67 and #70 are independent.What
Jira: MKT-22144
Consumer impact: none. Docs and
.gitignoreonly; neither file is in the publishedfileslist.CLAUDE.md(public-safe facts only, no internal workflow section or tracker keys): what the SDK is, stack and commands (including whynpm cineeds--legacy-peer-deps), the public-API contract (no breaking change; runtime change only for a confirmed bug fix, stated in the PR; host-only listeners, linking fix(security): accept extensionEvent only from window.parent #65; no data in console; current error-message text kept as the prefix), layout, review rules as ruleset 22725457 states them (tests run on PRs once ci: unit tests on pull requests (Node 24); untrack coverage; declare and prune dependencies #69 merges), Definition of Done.CONTRIBUTING.md: issues, security route viaSECURITY.md, PR steps, MIT licence for contributions..gitignore:.env,.env.*. No env sample: the library reads no environment variables.CODE_OF_CONDUCT.md, which needs an enforcement contact from the owners.Readiness items
routine.agents-md,dev.review-rules,sdlc.definition-of-done,legal.licence-eula(CONTRIBUTING part),security.env-not-trackedCommands (exit codes, Node 24 locally)
npm cinpm ci --legacy-peer-depsnpm run buildlint / typechecknpm testnpm audit --omit=devCommits went through the repo's hooks (GitGuardian pre-commit and pre-push); none bypassed. Local Snyk cannot authenticate (org permission / 401), so the PR's Snyk checks are the scan.
CI:
security-licensefails here as onmaster: it requires the current year inLICENSE(still 2019-2025). Fixed by #64 (DX-29607), not by this PR. All Snyk, GitGuardian, CodeQL and policy checks pass.🤖 Generated with Claude Code