Repository navigation
docs: CLAUDE.md (API contract, review rules, DoD) and CONTRIBUTING.md - #197
pritesh-jain-cstk wants to merge 3 commits into
Conversation
…G.md (MKT-22132) 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
🟠 High Severity - SLA Breached Issues (with fixes)Showing 6 issue(s) that have exceeded the 30-day SLA threshold:
🟡 Medium Severity - SLA Breached Issues (with fixes)Showing 10 issue(s) that have exceeded the 90-day SLA threshold:
ℹ️ Vulnerabilities Without Available Fixes (Informational Only)The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:
❌ BUILD FAILED - Security checks failed Please review and fix the security vulnerabilities before merging. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
🟠 High Severity - SLA Breached Issues (with fixes)Showing 6 issue(s) that have exceeded the 30-day SLA threshold:
🟡 Medium Severity - SLA Breached Issues (with fixes)Showing 10 issue(s) that have exceeded the 90-day SLA threshold:
ℹ️ Vulnerabilities Without Available Fixes (Informational Only)The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:
❌ BUILD FAILED - Security checks failed Please review and fix the security vulnerabilities before merging. |
Drop the internal workflow block, internal wiki links and Jira keys; known traps stated inline; listener rule links PR #193. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
🟠 High Severity - SLA Breached Issues (with fixes)Showing 6 issue(s) that have exceeded the 30-day SLA threshold:
🟡 Medium Severity - SLA Breached Issues (with fixes)Showing 10 issue(s) that have exceeded the 90-day SLA threshold:
ℹ️ Vulnerabilities Without Available Fixes (Informational Only)The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:
❌ BUILD FAILED - Security checks failed Please review and fix the security vulnerabilities before merging. |
Merge after #193: the CLAUDE.md rule on host-only listeners describes
mainonce #193 is in.Branch graph
Epic MKT-21672. This PR is highlighted. Only [Routine] depends on [Tests]; the rest are independent.
What
Jira: MKT-22132
Consumer impact: none. Built
mainand this branch and diffeddist/src/**/*.d.ts: no diff. Docs only; neither file is in the publishedfileslist.CLAUDE.md(public-safe: no internal workflow block, wiki links or Jira keys): stack and commands, the public-API contract (no breaking change; runtime change only for a confirmed bug fix, stated in the PR with the typings diff), layout, known traps stated inline, review rules as ruleset 22716531 states them (the unit-test workflow is not a required check), Definition of Done.CONTRIBUTING.md: issues, security route viaSECURITY.md, PR steps, MIT licence for contributions.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)Commands (exit codes, Node 24 locally)
npm cinpm run lintnpm run buildnpx tsc --noEmit -p .npm testnpm audit --omit=devCI:
security-scafails here as onmain: the lock pins axios 1.16.0 (8 High). That is fixed by #191 (MKT-22129 links it), not by this PR. The pre-push hook (npm test) passed on every push; none bypassed. Local Snyk cannot authenticate (SNYK-0005); the PR's Snyk checks are the scan.🤖 Generated with Claude Code