Repository navigation
DX | 02-11-2026 | Release - #287
Merged
Merged
Conversation
Back merge
…ules chore: Refactor documentation in AGENTS.md and remove redundant Cursor…
fix: Modified github workflows for release and remove check-branch workflow
Brings back-merge branch up to date with development so master can be back-merged cleanly (DX-20927). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
DX-20927 | Back-merge master into development
Publish on release:published from release.yml so the npm trusted publisher can be keyed on the filename, drop NODE_AUTH_TOKEN in favour of id-token: write (OIDC), run on Node 24 with npm@latest (trusted publishing needs npm >= 11.5.1), check out the release tag without persisted credentials. GitHub pre-releases go to the beta dist-tag. The existing guard that publishes only v-prefixed tags is kept. The GitHub Packages job gains the packages: write permission it was missing and publishes with the job's own token instead of a personal token. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The build-test job runs with the default read-only token, so the coverage report, sticky comment and test reporter fail with "Missing checks: write" even though the tests pass (red on every pull request since 2026-09-24). Grant the job just what those steps need, pin the coverage action to the commit the v2 tag resolves to today, and exempt the pinned file from Talisman's hex-string check. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Refuse to publish when the release tag is not "v" + the package.json version, so a tag cut on the wrong commit cannot publish an older version under a newer tag. The GitHub Packages job now uses the same beta/latest dist-tag rule as the npm job, so a GitHub pre-release never becomes latest on either registry. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
AGENTS.md and the dev-workflow skill still described npm-publish.yml on release: created with NPM_TOKEN / GIT_TOKEN. Publishing now runs from release.yml on release: published, npm through OIDC trusted publishing and GitHub Packages with the job token. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Snyk (dev, all-projects): critical 2 -> 0, high 76 -> 6, medium 33 -> 8, low 6 -> 1 npm audit: critical 4 -> 3, high 40 -> 33, moderate 9 -> 10, low 4 -> 0 .talismanrc: package-lock.json now ignores filecontent and base64content (sha512 integrity hashes; false positives). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…development chore: back-merge master into development
Lock file regenerated (fixed 1 on its own: fast-uri 3.1.8 -> 3.1.9). Direct upgrades: none (no patch or minor bump clears a finding). Overrides: none (library; remaining fixes are cross-major). Vuln count: snyk 0 -> 0, npm audit 6 -> 5. Build verified passing. Remaining (dev-only, need majors): jest-coverage-badges (minimist, mkdirp) and jest-junit 15 -> 17 (uuid). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
chore(release): publish to npm with trusted publishing
✅ 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 |
Coverage report
Test suite run success430 tests passing in 22 suites. Report generated by 🧪jest coverage report action from 0d7297e |
reeshika-h
previously approved these changes
Nov 2, 2026
…ection Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
dhaval-jain-cstk
previously approved these changes
Nov 2, 2026
ci: pass GitHub context to run steps via env vars (script injection)
reeshika-h
dismissed stale reviews from dhaval-jain-cstk and themself
via
November 2, 2026 12:36
0d7297e
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
reeshika-h
approved these changes
Nov 2, 2026
Coverage report
Test suite run success430 tests passing in 22 suites. Report generated by 🧪jest coverage report action from 0d7297e |
Aravind-Kumar-cstk
approved these changes
Nov 3, 2026
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.
Summary
Release PR: promotes
developmenttomaster. The main change is DX-27614 —release.ymlpublishes to npm with OIDC trusted publishing (noNPM_TOKEN). It has to be onmaster, where releases are cut from.What's included
release.yml(chore(release): publish to npm with trusted publishing #283): runs onrelease: published,id-token: write, Node 24 and latest npmpackage.json, and tags the GitHub Packages publishsrcpaths, coverage report permissions, release workflow fixesrelease.ymland trusted publishing;AGENTS.mdcleanupNotes
package.jsonis still1.9.1, so merging does not publish anything. Publishing needs a version bump and a GitHub release.🤖 Generated with Claude Code