Skip to content

ci: isolate secrets from pull request checks#63

Merged
Obiajulu-gif merged 1 commit into
Chainmove:mainfrom
Davidemulo:fix/issue-21-safe-ci
Jun 23, 2026
Merged

ci: isolate secrets from pull request checks#63
Obiajulu-gif merged 1 commit into
Chainmove:mainfrom
Davidemulo:fix/issue-21-safe-ci

Conversation

@Davidemulo

Copy link
Copy Markdown
Contributor

Summary

  • run lint, typecheck, and production builds for pull requests using only inert mock values
  • restrict repository secret injection to trusted pushes on main
  • remove private signing/deployment credentials and manual dispatch from CI
  • keep least-privilege permissions, npm caching, concurrency cancellation, and add job timeouts

Security

Fork PRs receive no repository secrets and run no deployment steps. Secret values are never echoed or printed.

Validation

  • npm ci
  • npm run lint (passes with 2 pre-existing warnings)
  • npm run typecheck
  • secret-free npm run build
  • workflow YAML parse and git diff --check

Closes #21

@Obiajulu-gif

Copy link
Copy Markdown
Collaborator

@Davidemulo great work, this look great

@Obiajulu-gif Obiajulu-gif merged commit 4be8854 into Chainmove:main Jun 23, 2026
2 checks passed
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.

[CI] Add GitHub Actions workflow for lint, typecheck, and build

2 participants