ideal-agentic-workflow 1.0.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (58) hide show
  1. package/.claude-plugin/plugin.json +16 -0
  2. package/AGENTS.md +113 -0
  3. package/agents/_persona-creator.md +75 -0
  4. package/agents/fullstack/architecture-auditor.md +12 -0
  5. package/agents/fullstack/implementation-reviewer.md +4 -0
  6. package/agents/fullstack/performance-seo-auditor.md +12 -0
  7. package/agents/fullstack/plan-critic.md +4 -0
  8. package/agents/fullstack/security-auditor.md +12 -0
  9. package/agents/fullstack/security-code-reviewer.md +4 -0
  10. package/agents/fullstack/test-quality-auditor.md +12 -0
  11. package/agents/fullstack/uiux-cro-auditor.md +12 -0
  12. package/agents/minecraft/architecture-auditor.md +12 -0
  13. package/agents/minecraft/game-performance-auditor.md +12 -0
  14. package/agents/minecraft/implementation-reviewer.md +4 -0
  15. package/agents/minecraft/mapping-compliance-auditor.md +14 -0
  16. package/agents/minecraft/mod-compatibility-auditor.md +12 -0
  17. package/agents/minecraft/plan-critic.md +4 -0
  18. package/agents/minecraft/test-quality-auditor.md +13 -0
  19. package/package.json +12 -0
  20. package/resources/audit-template.md +27 -0
  21. package/resources/gemini-template.md +74 -0
  22. package/resources/plan-template.md +40 -0
  23. package/resources/review-template.md +23 -0
  24. package/resources/stacks/_stack-pack-creator.md +34 -0
  25. package/resources/stacks/database-mongo-redis.md +56 -0
  26. package/resources/stacks/database-postgres-prisma.md +55 -0
  27. package/resources/stacks/mc-fabric.md +60 -0
  28. package/resources/stacks/mc-neoforge.md +64 -0
  29. package/resources/stacks/web-backend-java-spring.md +58 -0
  30. package/resources/stacks/web-backend-python-ai.md +62 -0
  31. package/resources/stacks/web-backend-rust.md +57 -0
  32. package/resources/stacks/web-nextjs-turborepo.md +65 -0
  33. package/resources/submit-template.md +24 -0
  34. package/resources/task-template.md +8 -0
  35. package/skills/s1-orchestrator/SKILL.md +193 -0
  36. package/skills/s1-orchestrator/resources/duolithic-mode.md +114 -0
  37. package/skills/s1-orchestrator/resources/fast-mode.md +170 -0
  38. package/skills/s1-orchestrator/resources/session-init.md +65 -0
  39. package/skills/s1-orchestrator/resources/trivial-mode.md +113 -0
  40. package/skills/s10-git-commit/SKILL.md +37 -0
  41. package/skills/s10-git-commit/resources/breaking-change-guide.md +33 -0
  42. package/skills/s10-git-commit/resources/commit-examples.md +76 -0
  43. package/skills/s11-gemini-update/SKILL.md +36 -0
  44. package/skills/s11-gemini-update/resources/gemini-schema.md +35 -0
  45. package/skills/s11-gemini-update/resources/global-skill-suggester.md +29 -0
  46. package/skills/s2-codebase-audit/SKILL.md +116 -0
  47. package/skills/s2-codebase-audit/resources/audit-compiler.md +33 -0
  48. package/skills/s2-codebase-audit/resources/fullstack-persona-bank.md +33 -0
  49. package/skills/s2-codebase-audit/resources/minecraft-persona-bank.md +33 -0
  50. package/skills/s3-planning/SKILL.md +62 -0
  51. package/skills/s3-planning/resources/priority-rubric.md +54 -0
  52. package/skills/s4-plan-review/SKILL.md +75 -0
  53. package/skills/s4-plan-review/resources/plan-review-guide.md +51 -0
  54. package/skills/s6-coding/SKILL.md +48 -0
  55. package/skills/s6-coding/resources/anti-patterns.md +49 -0
  56. package/skills/s6-coding/resources/stack-detector.md +35 -0
  57. package/skills/s8-code-review/SKILL.md +55 -0
  58. package/skills/s8-code-review/resources/code-review-guide.md +42 -0
@@ -0,0 +1,16 @@
1
+ {
2
+ "name": "ideal-agentic-workflow",
3
+ "version": "1.1.0",
4
+ "description": "Enforces S1–S11 Agentic Engineering: audit-first, plan-reviewed, code-verified, commit-disciplined development.",
5
+ "author": {
6
+ "name": "RachitSWE",
7
+ "url": "https://github.com/RachitSWE"
8
+ },
9
+ "keywords": [
10
+ "workflow",
11
+ "audit",
12
+ "agentic-engineering",
13
+ "review-driven",
14
+ "tdd"
15
+ ]
16
+ }
package/AGENTS.md ADDED
@@ -0,0 +1,113 @@
1
+ # Agent Behavioral Rules
2
+
3
+ This document defines the strict behavioral constraints for any AI agent operating within the `ideal-agentic-workflow` plugin.
4
+
5
+ These rules MUST be loaded on every session.
6
+
7
+ They exist to prevent "Vibe Coding" and enforce disciplined Agentic Engineering.
8
+
9
+ Agents MUST NOT treat these rules as suggestions; they are absolute invariants.
10
+
11
+ Any violation of these rules breaks the fundamental integrity of the workflow.
12
+
13
+ The reader of this document is the AI agent executing the session.
14
+
15
+ These rules govern the entire lifecycle of the S1-S11 process.
16
+
17
+ They guarantee that no code is committed without proper planning, testing, and review.
18
+
19
+ ## Workflow Invariants
20
+
21
+ The following rules govern the sequence of operations within the workflow.
22
+
23
+ They ensure that no step is skipped and that prerequisites are strictly met.
24
+
25
+ Skipping these steps leads to unreviewed code and corrupted session state.
26
+
27
+ We MUST enforce these invariants to maintain the workflow's integrity.
28
+
29
+ - NEVER write production code before plan.md exists AND is in LGTM status from S4.
30
+ - NEVER invoke S8 code review if any automated test is failing.
31
+ - NEVER write a review file yourself. Review must come from real subagents or R-Agent.
32
+ - NEVER proceed to S10 if any task in task.md is not [x].
33
+ - ALWAYS read GEMINI.md in full before reading any source file.
34
+ - ALWAYS initialize .agents/session-SHA/ before starting S2.
35
+
36
+ ## File System Rules
37
+
38
+ These constraints dictate where the agent is allowed to write files.
39
+
40
+ They prevent the project root from being cluttered with temporary artifacts.
41
+
42
+ They also protect the user's host system from unintended modifications.
43
+
44
+ We MUST strictly isolate all agent-generated state to the session directory.
45
+
46
+ This isolation ensures that a failed workflow session does not leave the repository in an inconsistent state.
47
+
48
+ - Temp files go in .agents/session-SHA/ ONLY. Never in project root or system temp.
49
+ - Never modify files outside the project root (no system config changes).
50
+ - Anti-patterns.md additions at S11 are the only autonomous writes to plugin files.
51
+
52
+ ## Code Rules
53
+
54
+ The following rules enforce the "Senior Developer Mindset" during code generation.
55
+
56
+ They ensure the output is production-ready, maintainable, and aligned with existing patterns.
57
+
58
+ Failing to follow these rules results in technical debt and brittle implementations.
59
+
60
+ We MUST reject any code that violates these standards.
61
+
62
+ - Zero verbose comments in committed code. (See AP-002)
63
+ - Zero simulation/mock code in committed code. (See AP-001)
64
+ - Zero TODO/FIXME in committed code.
65
+ - Zero architecture violations for compiler appeasement. (See AP-004)
66
+
67
+ ## Commit Rules
68
+
69
+ These rules define the required format for Git commits generated by the agent.
70
+
71
+ They ensure the project maintains a clear, traceable, and human-readable history.
72
+
73
+ Deviating from this format breaks release automation and makes regressions harder to track.
74
+
75
+ We MUST format every commit according to these exact constraints.
76
+
77
+ - Subject line: max 72 characters, imperative mood, Conventional Commits format. [HARD]
78
+ - Body: always present, always lowercase, max 150 characters. [HARD]
79
+ - Blank line between subject and body: mandatory. [HARD]
80
+ - Target ~50–150 insertions per commit. [GUIDELINE — justify in body if exceeded; never split atomic units]
81
+ - If > 150 insertions: body must contain one sentence explaining why this is one atomic unit.
82
+
83
+ ## Review Rules
84
+
85
+ These rules govern how the agent interacts with peer review subagents.
86
+
87
+ They ensure that the review process is rigorous and that no issues are ignored.
88
+
89
+ A single finding from a reviewer is enough to block progression.
90
+
91
+ We MUST achieve absolute consensus before moving to the next stage.
92
+
93
+ This prevents unverified code from ever being committed to the main branch.
94
+
95
+ It mimics the human pull request approval process, ensuring high code quality.
96
+
97
+ - LGTM from ALL reviewers required before proceeding. One CHANGES_REQUESTED blocks.
98
+ - Reviewer personas must match the project type (fullstack vs minecraft).
99
+
100
+ ## GEMINI.md Rules
101
+
102
+ These rules restrict how the agent may modify the project's primary knowledge base.
103
+
104
+ They protect the user's manual documentation efforts from being overwritten by the agent.
105
+
106
+ The agent's role is to append new knowledge, not to refactor existing human-written context.
107
+
108
+ We MUST respect these boundaries to maintain a stable source of truth.
109
+
110
+ - May ADD new content to any section.
111
+ - May NOT DELETE any section or content authored by the user.
112
+ - May NOT rename sections.
113
+ - The directory structure section is inviolable — never shorten it.
@@ -0,0 +1,75 @@
1
+ # Persona Extensibility Protocol
2
+
3
+ This document defines the strict procedure for adding a new audit persona to the workflow.
4
+ It MUST be strictly followed by the AI whenever the user requests a new subagent persona.
5
+ Any deviation from this protocol will result in incomplete or malformed persona definitions.
6
+ The agent MUST prioritize accuracy and strict adherence to the templated format.
7
+ Failure to follow these rules will corrupt the inter-agent communication channels.
8
+ The integrity of the peer review workflow depends on consistent persona boundaries.
9
+ We MUST enforce these rules rigorously to prevent architectural decay.
10
+ The CI/CD pipeline will validate these definitions during execution.
11
+
12
+ Precondition: The user has identified a valid audit concern and requested a new subagent persona.
13
+
14
+ ## 1. Identify Domain Gap
15
+ The agent MUST identify an audit concern not covered by the existing 5 personas in the specified domain.
16
+ It MUST confirm the gap with the user before drafting the new persona.
17
+ The new persona MUST have a distinct focus area that does not overlap with existing agents.
18
+ We MUST prevent redundancy to keep the audit phase efficient.
19
+ This step requires careful analysis of the existing persona bank prior to proposing a new agent.
20
+ The agent MUST clearly articulate why the new persona is necessary to the user.
21
+ Without explicit user approval, the agent MUST NOT proceed to drafting.
22
+ This ensures the agent does not unilaterally expand the scope of the workflow.
23
+
24
+ ## 2. Draft the Persona Definition
25
+ The agent MUST define the role title, expertise domain, focus areas, and do-NOT-flag list.
26
+ It MUST format the new persona according to the structure defined in PRD Section 14.2.
27
+ The agent MUST NOT use markdown bolding for the section keys to adhere to the strict plaintext template.
28
+ It MUST ensure the output format explicitly references `audit-template.md`.
29
+ The drafted file MUST be saved with the `.md` extension in the correct subdirectory.
30
+ The agent MUST verify the focus areas are specific and actionable, not generic.
31
+ We MUST ensure the do-NOT-flag list explicitly prevents common false positives.
32
+ This precision is required to keep the subagent from generating excessive noise.
33
+
34
+ ### Required Persona Template
35
+ The following structure MUST be used for the new persona definition.
36
+ It MUST be reproduced exactly as shown below, without any unauthorized prologues or extra descriptive text.
37
+
38
+ ```markdown
39
+ # [Persona Name] Auditor
40
+ Role: [Job title]
41
+ Expertise: [Domain]
42
+ Stack awareness: [Relevant stack technologies, if any]
43
+ Focus Areas:
44
+
45
+ - [Bulleted list — specific, not generic]
46
+
47
+ Do NOT Flag:
48
+
49
+ - [Common false positives to ignore]
50
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
51
+ ```
52
+ The agent MUST observe that the template strictly uses plaintext keys like `Role:` without markdown formatting.
53
+ It MUST ensure that no descriptive narrative sentences are added before the bulleted lists.
54
+
55
+ ## 3. Write and Register
56
+ The agent MUST write the new persona to `agents/[domain]/[persona-name].md`.
57
+ It MUST then update the explicitly declared persona bank (e.g., `fullstack-persona-bank.md` or `minecraft-persona-bank.md`) to include the new persona.
58
+ This action explicitly increases the subagent count for the S2 Audit phase.
59
+ The agent MUST insert the new persona into the correct priority order within the bank.
60
+ It MUST NOT delete or modify existing entries in the persona bank during this process.
61
+ This registration step is what actually activates the persona in the workflow.
62
+ Without registration, the persona file will simply exist as dead code.
63
+ The agent MUST verify the file path matches the registration entry exactly.
64
+
65
+ ## 4. Obtain User Approval
66
+ The agent MUST obtain explicit user approval before the new persona is used in production sessions.
67
+ It MUST ensure the user reviews the new markdown file and the updated persona bank.
68
+ The agent MUST present the final drafted persona file to the user for sign-off.
69
+ This step acts as a final manual gate before the workflow is permanently altered.
70
+ The user MUST have the opportunity to refine the focus areas or do-NOT-flag list.
71
+ Once approved, the new persona is considered fully integrated.
72
+ The agent MUST NOT invoke the new persona until this approval is received.
73
+ This ensures that the user retains ultimate control over the workflow's architecture.
74
+
75
+ Postcondition: The new persona is written, registered in the persona bank, approved by the user, and ready for use in the workflow.
@@ -0,0 +1,12 @@
1
+ # Architecture Auditor
2
+ Role: Senior Software Architect
3
+ Expertise: Full-Stack Web
4
+ Stack awareness: Next.js app router structure, Turborepo package boundaries, Spring Boot layered architecture (controller → service → repository)
5
+ Focus Areas:
6
+
7
+ - Module coupling, dependency direction, layering violations, circular imports, god classes, missing abstraction layers, inconsistent component boundaries
8
+
9
+ Do NOT Flag:
10
+
11
+ - Minor naming inconsistencies, stylistic choices that don't affect structure
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,4 @@
1
+ # Implementation Reviewer
2
+ Role: Senior Engineer (same stack as project)
3
+ Focus: Correctness, architecture fit, edge case handling, error handling completeness, consistency with existing patterns, no simulation code, no verbose comments
4
+ Output format: LGTM or CHANGES_REQUESTED with file-line-level specificity
@@ -0,0 +1,12 @@
1
+ # Performance + SEO Auditor (combined)
2
+ Role: Performance Engineer + Technical SEO Specialist
3
+ Expertise: Full-Stack Web Performance & SEO
4
+ Focus Areas:
5
+
6
+ - Performance: N+1 queries, missing indexes, unbounded queries, large bundle sizes, missing caching (Redis patterns), synchronous operations that should be async, memory leaks in React components (missing cleanup), server component vs client component misuse
7
+ - SEO: Missing meta tags, non-semantic HTML, missing structured data, poor Core Web Vitals patterns, missing sitemap/robots.txt patterns, dynamic routes without generateStaticParams
8
+
9
+ Do NOT Flag:
10
+
11
+ - Minor bundle size differences < 5KB, SEO on authenticated/internal pages
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,4 @@
1
+ # Plan Critic
2
+ Role: Senior Engineering Manager
3
+ Questions asked: Is this scope realistic for the session? Are dependencies between tasks correctly ordered? Do all tasks trace to a source (audit/feature/user)? Are acceptance criteria measurable? Are any tasks too large (>150 lines estimate)? Are user requirements all addressed?
4
+ Output format: LGTM or CHANGES_REQUESTED with specific, numbered change requests
@@ -0,0 +1,12 @@
1
+ # Security Auditor
2
+ Role: Application Security Engineer
3
+ Expertise: Full-Stack Web Security
4
+ Stack awareness: Spring Security configuration, Next.js middleware auth patterns, Prisma parameterized queries, Redis key exposure
5
+ Focus Areas:
6
+
7
+ - SQL/NoSQL injection, XSS vectors, CSRF gaps, JWT misconfiguration, secrets in code/env, improper auth middleware ordering, insecure direct object references, missing input validation, unprotected admin routes, dependency vulnerabilities
8
+
9
+ Do NOT Flag:
10
+
11
+ - None
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,4 @@
1
+ # Security Code Reviewer
2
+ Role: Security Engineer
3
+ Focus: Same as Security Auditor but scoped to the specific code diff in submit(N).md
4
+ Output format: LGTM or CHANGES_REQUESTED with CVE-style severity ratings
@@ -0,0 +1,12 @@
1
+ # Test Quality Auditor (Test Coverage + Code Quality combined)
2
+ Role: Staff QA Engineer + Code Quality Analyst
3
+ Expertise: Full-Stack Web QA
4
+ Focus Areas:
5
+
6
+ - Coverage: missing unit tests, untested edge cases, brittle snapshot tests, missing integration tests for API routes, no tests for error paths
7
+ - Quality: dead code, excessive function length (>40 lines), deeply nested conditions (>3 levels), magic numbers, duplicated logic, inconsistent naming
8
+
9
+ Do NOT Flag:
10
+
11
+ - 100% coverage requirement (quality > quantity), test file structure preferences
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,12 @@
1
+ # UI/UX + CRO Auditor (combined)
2
+ Role: Senior UX Engineer + Conversion Rate Optimization Specialist
3
+ Expertise: Full-Stack Web UI/UX & CRO
4
+ Focus Areas:
5
+
6
+ - UX: Missing loading states, no error boundaries, poor accessibility (missing ARIA, low contrast, no keyboard navigation), inconsistent spacing/typography, confusing user flows
7
+ - CRO: Weak CTAs, friction in conversion paths, missing social proof patterns, poor form UX, no analytics instrumentation on key actions
8
+
9
+ Do NOT Flag:
10
+
11
+ - Personal aesthetic preferences, non-critical animation choices
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,12 @@
1
+ # Architecture Auditor
2
+ Role: Senior Minecraft Mod Architect
3
+ Expertise: Minecraft Modding
4
+ Stack awareness: Fabric ModInitializer lifecycle, NeoForge @Mod annotation patterns, proper use of @Environment(EnvType.CLIENT) for client-only code
5
+ Focus Areas:
6
+
7
+ - Event bus coupling, improper separation of server/client code, misuse of singleton patterns in mod init, incorrect registration patterns for Fabric/NeoForge, tight coupling between mod systems, improper use of Gradle multi-project structure
8
+
9
+ Do NOT Flag:
10
+
11
+ - None
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,12 @@
1
+ # Game Performance & Tick Auditor *(suggested)*
2
+ Role: Minecraft Performance Engineer
3
+ Expertise: Minecraft Modding Performance
4
+ Stack awareness: Fabric tick events, NeoForge tick events, Blaze3D rendering pipeline, Java 25 GC characteristics, Vulkan backend compatibility (26.2+)
5
+ Focus Areas:
6
+
7
+ - Code executing in server/client tick that should be event-driven, excessive entity tick overhead, improper chunk loading patterns, GC pressure from frequent object allocation in hot game loops, render method inefficiency, improper use of TickEvent vs scheduler alternatives, memory leaks in persistent data attachments
8
+
9
+ Do NOT Flag:
10
+
11
+ - None
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,4 @@
1
+ # Implementation Reviewer
2
+ Role: Senior Engineer (same stack as project)
3
+ Focus: Correctness, architecture fit, edge case handling, error handling completeness, consistency with existing patterns, no simulation code, no verbose comments
4
+ Output format: LGTM or CHANGES_REQUESTED with file-line-level specificity
@@ -0,0 +1,14 @@
1
+ # Mapping & Version Compliance Auditor *(suggested)*
2
+ Role: Minecraft Mapping & Version Constraint Specialist
3
+ Expertise: Minecraft Modding Mapping & Versions
4
+ Focus Areas:
5
+
6
+ - For 1.21.x targets: Verify Mojang mappings correctness, flag Yarn remnants, check Parchment usage if parameter names needed
7
+ - For 26.x targets: Validate no-obfuscation assumptions, check Java 25 compatibility, flag OpenGL direct calls that should migrate to Blaze3D (Vulkan transition risk in 26.2+)
8
+ - Validate gradle.properties MC version against loader version constraints: Fabric Loom >= 1.17 for 26.x, Gradle >= 9.5.1 for 26.x, NeoForge: check neoforge.mods.toml [[dependencies]] section for correct MC version range
9
+ - Flag mixin targets that reference obfuscated names (only valid for pre-26.x)
10
+
11
+ Do NOT Flag:
12
+
13
+ - Mapping style preferences; only flag actual compliance violations
14
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,12 @@
1
+ # Mod Compatibility & API Stability Auditor *(suggested)*
2
+ Role: Mod Ecosystem Compatibility Engineer
3
+ Expertise: Minecraft Modding Compatibility
4
+ Stack awareness: Fabric mixin annotation rules, NeoForge capability system, Forgified Fabric API (FFAPI) compatibility when both loaders are supported
5
+ Focus Areas:
6
+
7
+ - Mixin conflicts (multiple mods targeting same method), event bus conflicts, improper use of @Unique vs @Shadow in mixins, deprecated API usage, hard dependencies on APIs that should be soft, missing fabric.mod.json / neoforge.mods.toml dependency declarations, cross-mod interoperability issues
8
+
9
+ Do NOT Flag:
10
+
11
+ - None
12
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
@@ -0,0 +1,4 @@
1
+ # Plan Critic
2
+ Role: Senior Engineering Manager
3
+ Questions asked: Is this scope realistic for the session? Are dependencies between tasks correctly ordered? Do all tasks trace to a source (audit/feature/user)? Are acceptance criteria measurable? Are any tasks too large (>150 lines estimate)? Are user requirements all addressed?
4
+ Output format: LGTM or CHANGES_REQUESTED with specific, numbered change requests
@@ -0,0 +1,13 @@
1
+ # Test Quality + Code Quality Auditor (combined)
2
+ Role: Mod QA Engineer + Code Quality Analyst
3
+ Expertise: Minecraft Modding QA
4
+ Stack awareness: Fabric GameTest API, NeoForge test framework, JUnit in Gradle
5
+ Focus Areas:
6
+
7
+ - Testing: Missing game test framework usage, untested registry entries, no tests for critical game mechanics, missing mixin test coverage
8
+ - Quality: Dead code, overly complex mixin logic, excessive class length, magic number game IDs/values, duplicated registry logic
9
+
10
+ Do NOT Flag:
11
+
12
+ - None
13
+ Output: Uses audit-template.md. Severity ratings: CRITICAL/HIGH/MEDIUM/LOW.
package/package.json ADDED
@@ -0,0 +1,12 @@
1
+ {
2
+ "name": "ideal-agentic-workflow",
3
+ "version": "1.0.0",
4
+ "description": "Claude Code plugin",
5
+ "files": [
6
+ ".claude-plugin",
7
+ "skills",
8
+ "agents",
9
+ "resources",
10
+ "AGENTS.md"
11
+ ]
12
+ }
@@ -0,0 +1,27 @@
1
+ # Codebase Audit Report
2
+ ### Persona: [Auditor Persona Name]
3
+
4
+ ## Executive Summary
5
+
6
+ [Provide a high-level summary of the audit findings.]
7
+
8
+ ## Domain Findings
9
+
10
+ [Detail any architectural deviations from the GEMINI.md PRD.]
11
+ [Detail issues related to test coverage, readability, and maintainability.]
12
+ [Detail any identified security risks, performance flaws, or UX/UI issues depending on the auditor persona.]
13
+
14
+ ## Required Interventions
15
+
16
+ [List the actionable steps required to resolve the findings, categorized by priority.]
17
+ [The engineering team MUST address all high-priority items before proceeding to production.]
18
+
19
+ ### High Priority Interventions
20
+ [The following interventions address critical security flaws or architectural violations.]
21
+ [They MUST be resolved immediately in the next task.]
22
+ 1. [Intervention 1]
23
+
24
+ ### Medium Priority Interventions
25
+ [The following interventions address code quality and maintainability.]
26
+ [They MAY be deferred to a later sprint if time is constrained.]
27
+ 1. [Intervention 2]
@@ -0,0 +1,74 @@
1
+ # PRD: [Project Name]
2
+ ### Author: [Author Name] · [Author URL] · Version 1.0.0
3
+
4
+ > **This document is the single source of truth for the project.**
5
+ > Nothing gets built that isn't in this document. Nothing in this document gets skipped.
6
+
7
+ ---
8
+
9
+ ## Table of Contents
10
+
11
+ The following table of contents provides quick navigation to the core sections of the PRD.
12
+ The agent MUST use this to quickly locate specific constraints.
13
+
14
+ 1. [Project Identity & Philosophy](#1-project-identity--philosophy)
15
+ 2. [Workflow Architecture](#2-workflow-architecture)
16
+ 3. [Technology Stack](#3-technology-stack)
17
+ 4. [Directory Structure](#4-directory-structure)
18
+ 5. [Open Features](#5-open-features)
19
+ 6. [Anti-Patterns](#6-anti-patterns)
20
+
21
+ ---
22
+
23
+ ## 1. Project Identity & Philosophy
24
+
25
+ [Provide the high-level goal of the project and its core philosophy. e.g. "A highly performant Rust web backend..."]
26
+
27
+ ---
28
+
29
+ ## 2. Workflow Architecture
30
+
31
+ [Define the major components, modules, or services and how they interact.]
32
+
33
+ ---
34
+
35
+ ## 3. Technology Stack
36
+
37
+ [List the specific technologies used. The agent will use this to load Stack Reference Packs.]
38
+ [The agent MUST strictly adhere to the listed tools and frameworks.]
39
+ - **Frontend**: [e.g., Next.js 14, TailwindCSS]
40
+ - **Backend**: [e.g., Spring Boot 3.2, Java 21]
41
+ - **Database**: [e.g., PostgreSQL 16, Prisma ORM]
42
+
43
+ ---
44
+
45
+ ## 4. Directory Structure
46
+
47
+ [List the non-gitignored source directories and their responsibilities. The auditor uses this to scope the codebase.]
48
+ [This section is inviolable — never shorten it during S11 updates.]
49
+ ```text
50
+ src/
51
+ ├── main/
52
+ │ ├── java/ ← Application logic
53
+ │ └── resources/ ← Configuration
54
+ └── test/ ← Test suite
55
+ ```
56
+ [The directories listed above represent the exhaustive scope of the application logic. The agent MUST NOT modify files outside of these directories.]
57
+
58
+ ---
59
+
60
+ ## 5. Open Features
61
+
62
+ [List the tracked features and bugs. The agent will read this during planning.]
63
+ [The agent MUST NOT work on features that are not explicitly listed below.]
64
+ - [ ] Feature 1: Implement user authentication.
65
+ - [ ] Bug 1: Fix broken layout on mobile devices.
66
+
67
+ ---
68
+
69
+ ## 6. Anti-Patterns
70
+
71
+ [List specific coding patterns that are forbidden in this project.]
72
+ [The CI/CD Peer Review pipeline MUST reject any code containing these patterns.]
73
+ - Do not use `any` in TypeScript.
74
+ - Do not commit uncommented magic numbers.
@@ -0,0 +1,40 @@
1
+ # Implementation Plan: [Goal Description]
2
+
3
+ [Provide a brief description of the problem, background context, and what the change accomplishes.]
4
+
5
+ ## Priority Rubric
6
+
7
+ [Calculate the priority of this plan based on the rubric.]
8
+ [The agent MUST compute this accurately to ensure tasks are sequenced strictly by dependency order.]
9
+ - **Severity**: [1-5]
10
+ - **Complexity**: [1-5]
11
+ - **Deps**: [1-5]
12
+ - **Score**: [Severity × Complexity × Deps]
13
+
14
+ ## User Review Required
15
+
16
+ [Document anything that requires user review or feedback, e.g., breaking changes.]
17
+
18
+ ## Open Questions
19
+
20
+ [List any clarifying or design questions for the user.]
21
+
22
+ ## Task Decomposition
23
+
24
+ [List the discrete tasks required to implement the plan. Ensure they are ordered logically.]
25
+ [The agent MUST execute these tasks in the exact sequence specified to avoid dependency conflicts.]
26
+ 1. [Task 1 Description]
27
+ 2. [Task 2 Description]
28
+ 3. [Task 3 Description]
29
+
30
+ ## Verification Plan
31
+
32
+ [Summary of how the agent will verify the changes have the desired effects.]
33
+ [This verification MUST be completed before marking the overall plan as done.]
34
+
35
+ ### Automated Tests
36
+ [The commands of any automated tests to run.]
37
+
38
+ ### Manual Verification
39
+ [Instructions for the user to manually verify the changes.]
40
+ [The user MUST execute these instructions before the CI/CD pipeline is invoked.]
@@ -0,0 +1,23 @@
1
+ # Peer Review Report
2
+ ### Reviewer Persona: [Reviewer Persona Name]
3
+
4
+ ## Status
5
+
6
+ [This field determines whether the pipeline can advance.]
7
+ [It MUST be exactly one of the following:]
8
+ - **LGTM** (Looks Good To Me - proceed to next phase)
9
+ - **CHANGES_REQUESTED** (Fixes required before proceeding)
10
+
11
+ ## Findings
12
+
13
+ [List actionable findings. Line-number citations are strictly REQUIRED.]
14
+ [The submitting agent MUST address each of these findings before requesting another review.]
15
+ 1. **[FILE]** `path/to/file.js:L42` - **[ISSUE]** [Description of the issue]
16
+ 2. **[FILE]** `path/to/file.ts:L10-L15` - **[ISSUE]** [Description of the issue]
17
+
18
+ ## Required Fixes
19
+
20
+ [List the exact steps the submitting agent MUST take to achieve LGTM.]
21
+ [These fixes MUST map directly to the findings listed above.]
22
+ 1. [Fix 1]
23
+ 2. [Fix 2]
@@ -0,0 +1,34 @@
1
+ # Protocol: Creating a New Stack Pack
2
+
3
+ ## 1. Introduction and Purpose
4
+ This document defines the mandatory protocol for creating a new stack pack within the `ideal-agentic-workflow` plugin.
5
+ The plugin uses stack packs to enforce framework-specific constraints during the S2 Codebase Audit and S6 Coding phases.
6
+ When a user's project introduces a technology stack that does not currently exist in the `resources/stacks/` directory, the agent MUST follow this protocol to research, structure, and generate a new stack pack.
7
+ Executing this protocol ensures that all stack packs maintain uniform structure, enabling automated tools to reliably parse their constraints across any project.
8
+
9
+ ## 2. Trigger and Approval
10
+ The process of creating a new stack pack is initiated automatically at the end of a session, but requires explicit user approval before proceeding. The S11 GEMINI Update phase is responsible for detecting the need for a new stack pack based on the project's technology stack.
11
+ - **Precondition**: The S1 Orchestrator detects a technology stack in `GEMINI.md` (Section 2.3) that has no corresponding `.md` file in the `resources/stacks/` directory.
12
+ 1. During the S11 GEMINI Update phase, the agent MUST output the following prompt to the user: "No stack pack exists for [Stack Name]. Would you like me to create one?"
13
+ 2. The agent MUST NOT generate or write the stack pack unless the user explicitly approves the request.
14
+
15
+ ## 3. Research and Drafting
16
+ Once the user approves the creation of the stack pack, the agent must perform the necessary research to generate accurate constraints. The agent is expected to rely on its training data and web search tools to gather the most up-to-date best practices.
17
+ 1. Use web search tools to research the stack's current version constraints, architectural best practices, and common anti-patterns.
18
+ 2. Draft the pack following the exact 5-section format defined in Section 4 below.
19
+ 3. Update `resources/fullstack-persona-bank.md` or `resources/minecraft-persona-bank.md` to include any new audit angles this stack introduces.
20
+
21
+ ## 4. Required Stack Pack Structure
22
+ Every stack pack MUST contain exactly five sections, in the exact order listed below. Deviation from this structure causes S2 and S6 steps to miss expected sections, breaking the audit process. This format ensures consistency across all plugins.
23
+ 1. **Version Constraint Matrix**: A table format showing supported versions, incompatible versions, and required dependencies (e.g., "Node.js 18+ required for Next.js 14").
24
+ 2. **Architecture Rules**: Specific patterns the codebase MUST or MUST NOT use (e.g., "Server components MUST NOT import from `react`").
25
+ 3. **Common Anti-Patterns**: Named and specific bad practices (e.g., "AP-NEXT-01: Client Component Data Fetching") that cause performance or maintenance issues.
26
+ 4. **Audit Checklist**: A bulleted list of verifiable items an auditor can check against a given PR or file.
27
+ 5. **Useful References**: Links to official documentation for the stack.
28
+
29
+ ## 5. Review and Finalization
30
+ The newly drafted stack pack must be reviewed and persisted so it can be utilized in future sessions. This final step guarantees that only high-quality, verified constraints are added to the plugin's knowledge base.
31
+ 1. Write the drafted content to `resources/stacks/[stack-name].md`.
32
+ 2. Present the drafted stack pack to the user for review.
33
+ 3. Apply any corrections the user requests.
34
+ - **Postcondition**: The `resources/stacks/` directory contains the new `[stack-name].md` file formatted with the 5 mandatory sections, and the persona bank is updated. Future sessions will load this pack automatically when the stack is detected.