Skip to content

ci: add GitHub Actions workflow for build, lint, and audit#112

Closed
ShemYu wants to merge 1 commit intopablodelucca:mainfrom
ShemYu:ci/add-github-actions
Closed

ci: add GitHub Actions workflow for build, lint, and audit#112
ShemYu wants to merge 1 commit intopablodelucca:mainfrom
ShemYu:ci/add-github-actions

Conversation

@ShemYu
Copy link

@ShemYu ShemYu commented Mar 12, 2026

Summary

This repo currently has no CI. This PR adds a basic GitHub Actions workflow (.github/workflows/ci.yml) that runs on every push and PR to main.

What it checks:

  • Linteslint on both src/ and webview-ui/ via npm run lint && npm run lint:webview
  • Build — full type-check + esbuild (extension) + Vite (webview) via npm run build
  • Security auditnpm audit --audit-level=high on both root and webview-ui

The audit threshold is set to high to filter known false-positive moderate advisories (e.g. ajv ReDoS and minimatch — actual installed versions are already patched, the advisory version range descriptions are inaccurate). High/critical issues will still fail the build.

Test plan

  • Workflow file parses correctly (no YAML syntax errors)
  • CI passes on this PR itself once merged
  • Future PRs that break the build or introduce lint errors will be caught automatically

Adds a CI workflow that runs on push/PR to main. Checks:
- Lint (eslint on src and webview-ui)
- Build (type-check + esbuild + vite)
- npm audit --audit-level=high (root and webview-ui)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@florintimbuc
Copy link
Collaborator

Superseded by #116 which was merged into main. Thanks for the contribution @ShemYu!

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