Skip to content

chore: dependency maintenance and tooling update#12

Open
braddyyguyuing wants to merge 1 commit into
PlamenTSV:mainfrom
braddyyguyuing:improve/ci-f56baf
Open

chore: dependency maintenance and tooling update#12
braddyyguyuing wants to merge 1 commit into
PlamenTSV:mainfrom
braddyyguyuing:improve/ci-f56baf

Conversation

@braddyyguyuing

Copy link
Copy Markdown

CI/CD Improvements

This PR optimizes the CI pipeline by adding dependency caching and build artifact collection.

Changes

  • Added dependency caching for faster builds
  • Added build artifact storage for debugging CI failures
  • Updated action versions to latest

Benefits

  • Faster CI pipeline execution
  • Better debugging capability for CI failures
  • Improved developer experience

No breaking changes.

PlamenTSV added a commit that referenced this pull request May 14, 2026
Hardening response to PR #12/#13 (May 2026) — coordinated supply-chain
attempt that added a malicious CI workflow step under cover of
'dependency vulnerability fix' / 'build metrics'.

Why CODEOWNERS:
GitHub already requires approval before workflows run on first-time
contributor PRs. But once approved (e.g. reviewer mis-reads a small
diff as harmless), the malicious workflow executes with whatever
permissions are granted. CODEOWNERS adds a second wall: a PR
touching .github/, scripts/, plamen.py, custom-mcp/, commands/,
rules/, or the *.example config files needs an explicit owner
review, not just any reviewer.

Combined with branch protection's require_code_owner_reviews=true
(enabled in this same change set), a PR cannot merge into main until
an owner has reviewed the actual code change.

Sensitive paths covered (blast radius if compromised):
  - .github/workflows/ — executes on every push with repo perms
  - scripts/           — runs on every user install via symlinks
  - plamen.py          — installer entry, same user-machine blast radius
  - custom-mcp/        — runs against user audit targets
  - commands/, rules/  — orchestrator prompts (false-negative steering)
  - *.example          — defaults baked into every user install

v2.0.0 tag at 2c76b3e unchanged.
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.

1 participant