Commit Graph

28 Commits

Author SHA1 Message Date
Chris Farhood a3bab704df Update SKILL.md 2026-05-14 07:38:30 -04:00
privilegedescalation-engineer[bot] c48eccd70c Update SDLC skill: add UAT_PLAYBOOK.md maintenance requirement (PRI-1487) 2026-05-14 04:16:25 +00:00
privilegedescalation-engineer[bot] ea1f585722 Rework UAT skill: remove per-plugin tables, reference UAT_PLAYBOOK.md 2026-05-14 04:15:34 +00:00
privilegedescalation-engineer[bot] bedef6ab6a remove test file 2026-05-14 04:14:49 +00:00
privilegedescalation-engineer[bot] 1fe4f900b0 test 2026-05-14 04:14:28 +00:00
privilegedescalation-qa[bot] 44e528c373 Add dedicated UAT skill with plugin testing procedures
Add dedicated UAT skill with plugin testing procedures
2026-05-14 03:15:29 +00:00
Chris Farhood c041da4847 Add dedicated UAT skill with plugin testing procedures
- Add skills/uat/SKILL.md with concrete testing procedures for all 7 Headlamp plugins
- Update SDLC skill to reference the new uat skill for detailed procedures
- Fix namespace reference: UAT runs in headlamp-uat namespace, not privilegedescalation

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-14 03:11:28 +00:00
Chris Farhood fe3b4b90d7 docs(product-context): add headlamp-argocd-plugin to plugin inventory
Plugin existed on GitHub but was missing from the skill inventory, causing
it to be omitted from UAT gap analysis. Count updated from 6 to 7.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-14 01:22:24 +00:00
Chris Farhood 40a8f3d773 Fix SDLC skill: require UUID stage/participant IDs, clarify Reviewers UI field
The previous commit used string IDs like "qa-review" for execution policy
stages, but the API requires UUIDs and rejects non-UUID values. Also
renamed the section to "Issue Reviewers and Approvers" to match the UI
field names that agents need to populate.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-12 00:04:46 +00:00
Chris Farhood b5aa2b54a0 Add 'Issue Handoff via Execution Policy' section to SDLC skill
- Added new section explaining how to configure executionPolicy for automated reviewer handoffs
- Documented Pipeline A execution policy with QA and UAT stages
- Documented Pipeline B execution policy with single QA stage
- Explained triggering handoffs via in_review status
- Referenced Paperclip API reference for full schema

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 23:31:52 +00:00
Chris Farhood bfe02545e5 Remove handoff protocol and status semantics from SDLC skill
These are Paperclip platform mechanics already covered by the
Paperclip skill. The SDLC skill should only contain development
process rules, not platform API usage patterns.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 21:45:04 +00:00
Chris Farhood 0641848c4b Overhaul SDLC to three-branch promotion model
Replace the 5-stage pipeline (CI→UAT→QA→CTO→CEO) with a three-branch
promotion chain: dev (engineer self-merge) → uat (QA gates) → main
(UAT gates). Removes CTO review stage, CEO merge bottleneck, and SLA
timelines. Each gate owner has merge authority.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 21:40:54 +00:00
Chris Farhood 40caf8cfee Remove SLA section from SDLC skill and diagram
SLA timelines are meaningless to agents operating in heartbeats.
Removed the 48-hour PR review SLA from SKILL.md and the SLA
gantt chart from sdlc-diagram.md.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 20:56:17 +00:00
Chris Farhood da86aa7754 Add SDLC pipeline Mermaid diagram
Visualizes both pipelines (A: plugin, B: infra), the board approval
gate, PR review SLA, handoff protocol, and issue status lifecycle.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 20:54:22 +00:00
privilegedescalation-engineer[bot] b1e2000542 fix: restore CI workflow with markdownlint config (#63)
- Restore .github/workflows/ci.yaml that was deleted in April cleanup
- Add .markdownlint.yaml with relaxed rules for skill files
- Fix MD040 error in skills/sdlc/SKILL.md (add language to code block)
- Allows line lengths > 80, emphasis-as-headings, compact tables

Fixes CI failures on 'Merge POLICIES.md content into agent instruction bundles' commit.

Co-authored-by: Chris Farhood <chris@farhood.org>
Co-authored-by: Paperclip <noreply@paperclip.ing>
2026-05-11 20:22:24 +00:00
Chris Farhood d4a6141986 Add non-negotiable rule: agents must never change other agents' model configs
Board directive (PRI-1245): agents suggesting or making model changes for
other agents due to quota exhaustion is explicitly forbidden. Quota issues
must be escalated to the board.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 19:30:03 +00:00
Chris Farhood d077c62bcb Improve CI health check script with enhanced monitoring
Enhanced the ci-health-check.sh script to:
- Add stale repo detection (repos with no updates in 30+ days)
- Add CI workflow configuration checks
- Add color-coded output for better readability
- Track multiple failure types (CI failures, stale repos, no CI)
- Provide clearer summary reporting
- Increase CRITICAL_THRESHOLD to 3 for better filtering

This enables proactive monitoring of both CI health and repository
maintenance status across all privilegedescalation repos.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 18:48:23 +00:00
Chris Farhood 8840bd874d Fix: Disable MD004 unordered list style rule in markdownlint
- Skill files use dashes for unordered lists, but markdownlint expects asterisks
- Disable MD004 to allow both dash and asterisk styles
- Aligns with existing exceptions for MD013, MD036, and MD060

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 18:33:02 +00:00
Chris Farhood 4c779823a0 Add CI health check script for automated failure detection
Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 18:28:09 +00:00
Chris Farhood 496be01898 fix: restore CI workflow with markdownlint config
- Restore .github/workflows/ci.yaml that was deleted in April cleanup
- Add .markdownlint.yaml with relaxed rules for skill files
- Fix MD040 error in skills/sdlc/SKILL.md (add language to code block)
- Allows line lengths > 80, emphasis-as-headings, compact tables

Fixes CI failures on 'Merge POLICIES.md content into agent instruction bundles' commit.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-11 18:28:09 +00:00
Chris Farhood 64269836f2 Merge pull request #59 from privilegedescalation/gandalf/safety-anti-impersonation-rules 2026-05-09 12:43:01 -04:00
Chris Farhood a03256c231 Update safety skill: add anti-impersonation and role-boundary rules
Following PRI-737 investigation, add two rules to skills/safety/SKILL.md:

1. Anti-impersonation rule: agents must never sign, attribute, or present
   GitHub comments, PR reviews, or external communications as another
   agent. Every comment must accurately identify the authoring agent.

2. Role-boundary rule for GitHub actions: agents must only post GitHub PR
   comments and reviews within their defined SDLC role (engineer, QA, UAT,
   CTO, CEO). An agent must not post a review type belonging to another
   role, even if that role's agent has not yet completed its review.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-09 16:16:22 +00:00
Chris Farhood 1ebc0b0d89 Merge pull request #62 from privilegedescalation/countess/agent-process-review 2026-05-09 11:46:01 -04:00
Chris Farhood 6930b7a258 Optimize SDLC and coding-standards skills, remove duplication
SDLC skill (250 → ~127 lines):
- Remove Hugh-exclusive .github/workflows/ language; engineers share access
- Condense 48-hour SLA from 38 to 8 lines
- Replace verbose 5-stage pipeline description with compact diagrams
- Condense handoff protocol from 17 to 5 lines
- Remove status transition rules table (redundant with handoff protocol)
- Remove agent roster (agents have UUIDs in their own AGENTS.md)
- Remove work distribution section (redundant with agent instructions)

Coding-standards skill:
- Add SemVer, ArtifactHub distribution, ghcr.io registry rules
- Add Renovate/Dependabot, no-package-mirrors, npm-audit rules
- These were previously only in individual AGENTS.md files

Part of PRI-1094 — agent and process review.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-09 15:29:08 +00:00
Chris Farhood d69f5e4bd4 cleanup 2026-05-09 07:53:46 -07:00
privilegedescalation-ceo[bot] b7335c078e Merge pull request #61 from privilegedescalation/feat/product-context-skill
Add product-context skill
2026-05-09 02:42:34 +00:00
Chris Farhood 8b13f024e5 Add product-context skill extracted from Karen's AGENTS.md
Extracts the product context section (plugin portfolio, target users,
competitive landscape, evaluation framework, feature spec template)
into a version-controlled company skill at skills/product-context/SKILL.md.
Updates CLAUDE.md with skill documentation and loading order.

Part of PRI-1094 — agent and process review.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-05-09 00:50:27 +00:00
Chris Farhood 12ccf82454 Revise PR review SLA: remove threat language, focus on visibility and process
Replace dismissal-threat framing with operational consequences:
- 24h: public visibility + status flag
- 48h: merge queue block + escalation
- 72h+: blocks release if critical-path
- Exceptions: documented hand-off, not absolute prohibition

This makes the enforcement mechanism work for agents (visibility/process blocking)
rather than humans (dismissal threats), matching actual organizational incentives.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-05 10:53:13 +00:00