Skip to content

feat: publish smart-diff as a GitHub Action (smart-diff-ai) - #79

Merged
mcarvin8 merged 2 commits into
mainfrom
feat/github-action
Aug 25, 2026
Merged

mcarvin8 merged 2 commits into
mainfrom
feat/github-action

Conversation

@mcarvin8

Copy link
Copy Markdown
Owner

Summary

  • Ships this package as a native GitHub Action (action.yml, node24 runtime, no Docker/Salesforce CLI needed) via a pre-bundled dist/action/index.cjs (esbuild) — same pattern used for apex-tests-git-delta.
  • Marketplace listing name is smart-diff-ai (smart-diff is already taken on the Marketplace); provider credentials are picked up from the environment exactly as the CLI does today (env: block in the consuming workflow).
  • CI enforces the committed Action bundle stays in sync with source (test.yml), a smoke-test workflow exercises the Action against a fixture repo (bad-ref and real-diff paths), and release.yml now moves a floating major-version tag (e.g. v6) on every release for uses: mcarvin8/smart-diff@v6 consumers.

Test plan

  • npm run build:action produces dist/action/index.cjs and it runs end-to-end locally (verified against a real diff + real provider error surfacing correctly)
  • npm test — 370 tests passing, 100% statement/line/function coverage, 98.62% branch coverage (thresholds are 90%)
  • npm run lint / npm run lint:dependencies — clean
  • npm ci verified clean against the updated lockfile
  • First real release will validate the floating major-tag job and the Marketplace listing itself

🤖 Generated with Claude Code

Ships this package as a native GitHub Action so consumers get PR/commit
diff summaries without installing the CLI. Runs on node24 from a
pre-bundled dist/action/index.cjs (esbuild), same pattern used for
apex-tests-git-delta. Marketplace name is "smart-diff-ai" since
"smart-diff" is already taken.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcarvin8
mcarvin8 merged commit 6323dad into main Aug 25, 2026
7 checks passed
@mcarvin8
mcarvin8 deleted the feat/github-action branch August 25, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants