Skip to content

chore: add security workflow and update gitignore to exclude sensitiv…#50

Merged
jlc254 merged 7 commits into
mainfrom
dev
Apr 16, 2026
Merged

chore: add security workflow and update gitignore to exclude sensitiv…#50
jlc254 merged 7 commits into
mainfrom
dev

Conversation

@jumalaw98
Copy link
Copy Markdown
Contributor

@jumalaw98 jumalaw98 commented Apr 16, 2026

… the environment and credential files


Summary by cubic

Added a security workflow that runs on all pushes, blocks committed .env files (and secret-like templates), and runs heavy scans only on dev/main. Removed a checked-in .env and expanded .gitignore to cover more secrets and local data.

  • New Features

    • GitHub Actions: trigger on all pushes; run audit/codeql only on dev/main; always run check-env to fail on tracked .env and validate .env.* templates for real keys.
    • .gitignore/CI: ignore .env, .env.* (keep !.env.example), *.local, *.pem, *.key, *.crt, *.sqlite, *.dump, *.secret, secrets.json, auth.json, supabase/.temp/, supabase/.branches/; semantic PRs now allow security type.
  • Migration

    • If a .env is tracked, remove it: git rm --cached ; ensure .env.example only has placeholders.

Written for commit d84fff2. Summary will update on new commits.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sytcolabs Ready Ready Preview, Comment Apr 16, 2026 3:58pm

@github-actions github-actions Bot added ci CI/CD pipeline updates chore labels Apr 16, 2026
@github-actions
Copy link
Copy Markdown

Changelog preview

This is what the changelog entry will look like once this PR is merged.

No conventional commits found in this PR.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 issues found across 3 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name=".github/workflows/security.yml">

<violation number="1" location=".github/workflows/security.yml:193">
P1: This check only catches files named exactly `.env`, so tracked secret-bearing variants like `.env.local` or `.env.production` will slip through.</violation>
</file>

<file name=".gitignore">

<violation number="1" location=".gitignore:28">
P2: `.env*` also ignores `.env.example`, which this repo's setup docs rely on as the committed environment template.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread .github/workflows/security.yml Outdated
Comment thread .gitignore Outdated
@sonarqubecloud
Copy link
Copy Markdown

@jlc254 jlc254 merged commit e11a9d3 into main Apr 16, 2026
32 of 33 checks passed
@jlc254 jlc254 deleted the dev branch April 16, 2026 16:11
@jlc254 jlc254 restored the dev branch April 16, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved chore ci CI/CD pipeline updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants