@ryuenn3123/agentic-senior-core 5.7.0 → 5.8.1
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.
- package/.agents/plugins/agentic-senior-core/rules/agentic-senior-core.md +7 -0
- package/.agents/plugins/agentic-senior-core/skills/asc/SKILL.md +4 -1
- package/.agents/plugins/agentic-senior-core/skills/asc-add-feature/SKILL.md +46 -0
- package/.agents/plugins/agentic-senior-core/skills/asc-audit/SKILL.md +2 -0
- package/.agents/plugins/agentic-senior-core/skills/asc-debt/SKILL.md +2 -0
- package/.agents/plugins/agentic-senior-core/skills/asc-new-project/SKILL.md +49 -0
- package/.agents/plugins/agentic-senior-core/skills/asc-refactor/SKILL.md +23 -2
- package/.agents/plugins/agentic-senior-core/skills/asc-reference/SKILL.md +2 -0
- package/.agents/plugins/agentic-senior-core/skills/asc-review/SKILL.md +2 -0
- package/.agents/rules/agentic-senior-core.md +7 -0
- package/.claude-plugin/plugin.json +1 -1
- package/.clinerules/agentic-senior-core.md +7 -0
- package/.codex-plugin/plugin.json +1 -1
- package/.continue/rules/agentic-senior-core.md +7 -0
- package/.cursor/rules/agentic-senior-core.mdc +7 -0
- package/.devin/rules/agentic-senior-core.md +7 -0
- package/.devin-plugin/plugin.json +1 -1
- package/.github/copilot-instructions.md +7 -0
- package/.github/plugin/plugin.json +1 -1
- package/.kilocode/rules/agentic-senior-core.md +7 -0
- package/.kiro/steering/agentic-senior-core.md +7 -0
- package/.openclaw/skills/asc/SKILL.md +4 -8
- package/.openclaw/skills/asc-adapter/SKILL.md +37 -0
- package/.openclaw/skills/asc-add-feature/SKILL.md +46 -0
- package/.openclaw/skills/asc-audit/SKILL.md +2 -0
- package/.openclaw/skills/asc-debt/SKILL.md +2 -7
- package/.openclaw/skills/asc-new-project/SKILL.md +49 -0
- package/.openclaw/skills/asc-refactor/SKILL.md +56 -0
- package/.openclaw/skills/asc-reference/SKILL.md +2 -7
- package/.openclaw/skills/asc-review/SKILL.md +2 -0
- package/.openhands/microagents/agentic-senior-core.md +7 -0
- package/.roo/rules/agentic-senior-core.md +7 -0
- package/.windsurf/rules/agentic-senior-core.md +7 -0
- package/.zed/rules/agentic-senior-core.md +7 -0
- package/AGENTS.md +7 -0
- package/CONVENTIONS.md +7 -0
- package/README.md +80 -6
- package/commands/asc-add-feature.md +1 -0
- package/commands/asc-add-feature.toml +6 -0
- package/commands/asc-help.md +2 -0
- package/commands/asc-help.toml +11 -1
- package/commands/asc-new-project.md +1 -0
- package/commands/asc-new-project.toml +6 -0
- package/gemini-extension.json +1 -1
- package/hooks/path-util.cjs +23 -0
- package/hooks/post-edit-enforce.js +34 -0
- package/package.json +2 -2
- package/plugin.yaml +1 -1
- package/scripts/mcp-server/tools.mjs +10 -0
- package/skills/asc/SKILL.md +4 -1
- package/skills/asc-add-feature/SKILL.md +46 -0
- package/skills/asc-audit/SKILL.md +2 -0
- package/skills/asc-debt/SKILL.md +2 -0
- package/skills/asc-new-project/SKILL.md +49 -0
- package/skills/asc-refactor/SKILL.md +23 -2
- package/skills/asc-reference/SKILL.md +2 -0
- package/skills/asc-review/SKILL.md +2 -0
|
@@ -57,6 +57,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
57
57
|
|
|
58
58
|
For domain-specific rules (Testing, API Design, Database, Frontend, Infrastructure, Resilience), use `/asc-reference`.
|
|
59
59
|
|
|
60
|
+
## Workflow
|
|
61
|
+
|
|
62
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
63
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
64
|
+
they set up a research/plan gate before implementation. Let the user
|
|
65
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
66
|
+
|
|
60
67
|
## Response Style
|
|
61
68
|
|
|
62
69
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Follow with context only when the action depends on it.
|
|
@@ -4,6 +4,8 @@ Universal AI coding rules. Write code like a staff engineer.
|
|
|
4
4
|
|
|
5
5
|
## Available Commands
|
|
6
6
|
|
|
7
|
+
- `/asc-new-project` -- Greenfield workflow (Define -> Spec -> Implement -> Validate)
|
|
8
|
+
- `/asc-add-feature` -- Brownfield workflow (Research -> Plan -> Implement)
|
|
7
9
|
- `/asc-refactor` -- Structured refactoring workflow with pre-checks and validation
|
|
8
10
|
- `/asc-review` -- Production-risk code review with severity-ordered findings
|
|
9
11
|
- `/asc-audit` -- Security and architecture audit
|
|
@@ -17,8 +19,9 @@ On plugin-tier hosts (Claude Code, Codex CLI, Copilot CLI), a PostToolUse hook f
|
|
|
17
19
|
- New dependencies against stdlib duplicates (decision ladder step 3)
|
|
18
20
|
- LOC delta > 30 lines on edits (step 5)
|
|
19
21
|
- New files > 50 lines (steps 1–2)
|
|
22
|
+
- Workflow gate violations: source/config edits during research or plan phases
|
|
20
23
|
|
|
21
|
-
Violations inject a nudge referencing the specific ladder step. The hook is silent when no issues are found.
|
|
24
|
+
Violations inject a nudge referencing the specific ladder step or workflow phase. The hook is silent when no issues are found. All enforcement is advisory — nudges, not hard blocks.
|
|
22
25
|
|
|
23
26
|
## What It Does
|
|
24
27
|
|
|
@@ -0,0 +1,46 @@
|
|
|
1
|
+
# Add Feature Workflow
|
|
2
|
+
|
|
3
|
+
Structured brownfield workflow. Adapted from QRSPI to prevent context rot and ensure alignment before building.
|
|
4
|
+
|
|
5
|
+
Grounded in: RPI (Dex Horthy, HumanLayer 2025) with corrections from QRSPI 8-stage evolution (Coding Agents Conference, March 2026). Plan-reading illusion fix and instruction budget constraint applied. Stages 2/5/7 adapted; stages 1/3/4/6/8 skipped as too heavyweight for individual-developer workflow.
|
|
6
|
+
|
|
7
|
+
## Gate Mechanism
|
|
8
|
+
|
|
9
|
+
This workflow nudges the agent to stop at each phase boundary, same enforcement tier as the existing decision ladder — not a hard block. Bypasses are logged to the debt ledger.
|
|
10
|
+
|
|
11
|
+
**Known limitation:** Bypass-to-debt-ledger logging is self-reported by the agent, not enforced by the hook. The PostToolUse hook has no MCP access — it nudges the agent to log, but cannot write the debt entry itself.
|
|
12
|
+
|
|
13
|
+
To track phase, write to `workflow-gate.json` via the `state_write` MCP tool.
|
|
14
|
+
Format:
|
|
15
|
+
```json
|
|
16
|
+
{
|
|
17
|
+
"workflow": "asc-add-feature",
|
|
18
|
+
"phase": "<current_phase>",
|
|
19
|
+
"updatedAt": "<ISO-timestamp>"
|
|
20
|
+
}
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
## Phase 1: Research (No Code Changes)
|
|
24
|
+
|
|
25
|
+
1. Write `workflow-gate.json` with phase `research`.
|
|
26
|
+
2. Map existing code: patterns, utilities, dependencies already in use.
|
|
27
|
+
3. Identify what must NOT be rebuilt (e.g., existing validation helpers).
|
|
28
|
+
4. Output a factual research summary.
|
|
29
|
+
5. **STOP and wait for user approval.** Do not plan or implement.
|
|
30
|
+
|
|
31
|
+
## Phase 2: Plan
|
|
32
|
+
|
|
33
|
+
1. On approval of Phase 1, update `workflow-gate.json` phase to `plan`.
|
|
34
|
+
2. Create a numbered, step-by-step implementation plan with specific files, functions, and line references.
|
|
35
|
+
3. Include a "Don't Build" list from the research phase.
|
|
36
|
+
4. **Callout: Plan-Reading Illusion.** Ask the user to explicitly verify the plan against the codebase, not just skim it.
|
|
37
|
+
5. Output the plan.
|
|
38
|
+
6. **STOP and wait for user approval.** Do not implement.
|
|
39
|
+
|
|
40
|
+
## Phase 3: Implement
|
|
41
|
+
|
|
42
|
+
1. On approval of Phase 2, update `workflow-gate.json` phase to `implement`.
|
|
43
|
+
2. Recommend a fresh context (intentional compaction) if the context window is getting full.
|
|
44
|
+
3. Execute the approved plan.
|
|
45
|
+
4. Validate: tests pass, no duplicate code introduced, plan items checked off.
|
|
46
|
+
5. On completion, clear the state in `workflow-gate.json` by overwriting it with `{}`.
|
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
Security and architecture audit. Deeper than review, focused on finding vulnerabilities and structural anti-patterns.
|
|
4
4
|
|
|
5
|
+
Grounded in: OWASP Top 10 (2021), OWASP ASVS v4, CVSS vulnerability report structure, CWE classification.
|
|
6
|
+
|
|
5
7
|
## Audit Scope
|
|
6
8
|
|
|
7
9
|
1. **Trust boundaries**: Every point where external input enters the system. Validate that inputs are sanitized, normalized, and rejected when invalid.
|
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
Track deferred enforcement violations. When an ASC ladder nudge fires and the shortcut is accepted rather than fixed, log it here for later resolution.
|
|
4
4
|
|
|
5
|
+
Grounded in: Cunningham's technical debt metaphor (1992). Entry format (ladder step + violation + status lifecycle) is an ASC-specific implementation.
|
|
6
|
+
|
|
5
7
|
## Storage
|
|
6
8
|
|
|
7
9
|
Entries persist in `.agent-context/state/debt-ledger.json` via MCP `state_read`/`state_write`. The ledger is a JSON array of entry objects.
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
# New Project Workflow
|
|
2
|
+
|
|
3
|
+
Structured greenfield workflow. Prevents building before alignment on what to build.
|
|
4
|
+
|
|
5
|
+
Grounded in: Spec-Driven Development (SDD) with scaffolding-spec approach. Specs guide implementation, then the code becomes the source of truth — specs are not maintained as living documents unless the team explicitly opts in.
|
|
6
|
+
|
|
7
|
+
## Gate Mechanism
|
|
8
|
+
|
|
9
|
+
This workflow nudges the agent to stop at each phase boundary, same enforcement tier as the existing decision ladder — not a hard block. Bypasses are logged to the debt ledger.
|
|
10
|
+
|
|
11
|
+
**Known limitation:** Bypass-to-debt-ledger logging is self-reported by the agent, not enforced by the hook. The PostToolUse hook has no MCP access — it nudges the agent to log, but cannot write the debt entry itself.
|
|
12
|
+
|
|
13
|
+
To track phase, write to `workflow-gate.json` via the `state_write` MCP tool.
|
|
14
|
+
Format:
|
|
15
|
+
```json
|
|
16
|
+
{
|
|
17
|
+
"workflow": "asc-new-project",
|
|
18
|
+
"phase": "<current_phase>",
|
|
19
|
+
"updatedAt": "<ISO-timestamp>"
|
|
20
|
+
}
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
## Phase 1: Define (No Code)
|
|
24
|
+
|
|
25
|
+
1. Write `workflow-gate.json` with phase `research`.
|
|
26
|
+
2. Clarify with the user: tech stack, target platform, constraints, non-goals.
|
|
27
|
+
3. Propose a directory structure and module boundaries.
|
|
28
|
+
4. Output a project brief summarizing decisions.
|
|
29
|
+
5. **STOP and wait for user approval.** Do not write specs or code.
|
|
30
|
+
|
|
31
|
+
## Phase 2: Spec (No Implementation Code)
|
|
32
|
+
|
|
33
|
+
1. On approval of Phase 1, update `workflow-gate.json` phase to `plan`.
|
|
34
|
+
2. Write per-feature specs with acceptance criteria and edge cases.
|
|
35
|
+
3. Specs are scaffolding — they guide the build, then the code is the source of truth.
|
|
36
|
+
4. Output specs for review.
|
|
37
|
+
5. **STOP and wait for user approval.** Do not implement.
|
|
38
|
+
|
|
39
|
+
## Phase 3: Implement
|
|
40
|
+
|
|
41
|
+
1. On approval of Phase 2, update `workflow-gate.json` phase to `implement`.
|
|
42
|
+
2. Build against the approved specs. Apply the ASC decision ladder on every file.
|
|
43
|
+
3. Run the decision ladder: does this need to exist? Does stdlib cover it? One function or full module?
|
|
44
|
+
|
|
45
|
+
## Phase 4: Validate
|
|
46
|
+
|
|
47
|
+
1. Run tests. Check each spec's acceptance criteria.
|
|
48
|
+
2. Confirm nothing was over-built beyond what the specs required.
|
|
49
|
+
3. On completion, clear the state in `workflow-gate.json` by overwriting it with `{}`.
|
|
@@ -2,12 +2,33 @@
|
|
|
2
2
|
|
|
3
3
|
Structured refactoring workflow. Preserves existing behavior while improving structure.
|
|
4
4
|
|
|
5
|
+
Grounded in: Fowler's Refactoring (read-understand-smallest scope-preserve behavior), Rule of Three (abstraction threshold), YAGNI principle (XP/Kent Beck). Empirical evidence: agents dominate low-level refactors (rename, extract, type changes) but struggle with multi-file architectural changes (arXiv, 15k+ instance study).
|
|
6
|
+
|
|
7
|
+
## YAGNI Scan (Before Any Restructuring)
|
|
8
|
+
|
|
9
|
+
Before changing structure, scan for speculative code to remove:
|
|
10
|
+
- "Just in case" logic, unused feature flags, dead branches behind config toggles.
|
|
11
|
+
- Abstractions wrapping a single implementation with no second consumer.
|
|
12
|
+
- Premature extension points (plugin hooks, strategy patterns) with one concrete path.
|
|
13
|
+
|
|
14
|
+
Removing speculative code is the highest-value, lowest-risk refactor type.
|
|
15
|
+
|
|
16
|
+
## Classify Before Proceeding
|
|
17
|
+
|
|
18
|
+
After reading the target code, classify the refactor:
|
|
19
|
+
|
|
20
|
+
- **Low-level** (rename, extract method, type safety, dead code removal): proceed after explaining the change. No approval gate needed.
|
|
21
|
+
- **High-level** (move module, change architecture, redesign abstractions, multi-file structural changes): output a plan with specific files and rationale, then **stop and wait for user approval** before implementing.
|
|
22
|
+
|
|
23
|
+
**Known limitation:** The high-level refactor gate is a skill-text instruction — the agent self-classifies and self-stops. Unlike `/asc-add-feature` and `/asc-new-project`, this gate is not backed by the PostToolUse hook or workflow-gate.json. If the agent bypasses the gate, there is no automated nudge.
|
|
24
|
+
|
|
5
25
|
## Before Editing
|
|
6
26
|
|
|
7
27
|
1. Read the target code and understand existing patterns.
|
|
8
28
|
2. Identify the smallest relevant scope for the refactor.
|
|
9
|
-
3.
|
|
10
|
-
4. If the change touches
|
|
29
|
+
3. Classify the refactor (see above). If high-level, stop and present a plan.
|
|
30
|
+
4. If the change touches UI, check accessibility and responsive behavior.
|
|
31
|
+
5. If the change touches dependencies, verify current official docs.
|
|
11
32
|
|
|
12
33
|
## Refactor Rules
|
|
13
34
|
|
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
Domain-specific coding rules for testing, API design, database queries, frontend components, infrastructure configs, and service resilience. Load this skill when working on any of these domains.
|
|
4
4
|
|
|
5
|
+
Grounded in: WCAG 2.2 AA (accessibility), Fowler's Money Pattern (monetary types), Nygard's Release It! (resilience patterns), AWS Well-Architected Reliability Pillar (backoff/circuit breakers), OWASP Secure Coding Practices (input validation, logging).
|
|
6
|
+
|
|
5
7
|
## Testing
|
|
6
8
|
|
|
7
9
|
- Write tests for business logic and boundary failures, not implementation details.
|
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
Production-risk code review. Prioritize findings by severity.
|
|
4
4
|
|
|
5
|
+
Grounded in: OWASP Risk Rating Methodology, Google Engineering Practices (code review guide), Clean Architecture (transport vs business layer separation).
|
|
6
|
+
|
|
5
7
|
## Before Reviewing
|
|
6
8
|
|
|
7
9
|
1. Read the changed files and understand the scope.
|
|
@@ -57,6 +57,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
57
57
|
|
|
58
58
|
For domain-specific rules (Testing, API Design, Database, Frontend, Infrastructure, Resilience), use `/asc-reference`.
|
|
59
59
|
|
|
60
|
+
## Workflow
|
|
61
|
+
|
|
62
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
63
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
64
|
+
they set up a research/plan gate before implementation. Let the user
|
|
65
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
66
|
+
|
|
60
67
|
## Response Style
|
|
61
68
|
|
|
62
69
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Follow with context only when the action depends on it.
|
|
@@ -86,6 +86,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
86
86
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
87
87
|
- Circuit breakers for unhealthy dependencies.
|
|
88
88
|
|
|
89
|
+
## Workflow
|
|
90
|
+
|
|
91
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
92
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
93
|
+
they set up a research/plan gate before implementation. Let the user
|
|
94
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
95
|
+
|
|
89
96
|
## Response Style
|
|
90
97
|
|
|
91
98
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -86,6 +86,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
86
86
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
87
87
|
- Circuit breakers for unhealthy dependencies.
|
|
88
88
|
|
|
89
|
+
## Workflow
|
|
90
|
+
|
|
91
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
92
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
93
|
+
they set up a research/plan gate before implementation. Let the user
|
|
94
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
95
|
+
|
|
89
96
|
## Response Style
|
|
90
97
|
|
|
91
98
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -91,6 +91,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
91
91
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
92
92
|
- Circuit breakers for unhealthy dependencies.
|
|
93
93
|
|
|
94
|
+
## Workflow
|
|
95
|
+
|
|
96
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
97
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
98
|
+
they set up a research/plan gate before implementation. Let the user
|
|
99
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
100
|
+
|
|
94
101
|
## Response Style
|
|
95
102
|
|
|
96
103
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -86,6 +86,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
86
86
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
87
87
|
- Circuit breakers for unhealthy dependencies.
|
|
88
88
|
|
|
89
|
+
## Workflow
|
|
90
|
+
|
|
91
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
92
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
93
|
+
they set up a research/plan gate before implementation. Let the user
|
|
94
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
95
|
+
|
|
89
96
|
## Response Style
|
|
90
97
|
|
|
91
98
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -86,6 +86,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
86
86
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
87
87
|
- Circuit breakers for unhealthy dependencies.
|
|
88
88
|
|
|
89
|
+
## Workflow
|
|
90
|
+
|
|
91
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
92
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
93
|
+
they set up a research/plan gate before implementation. Let the user
|
|
94
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
95
|
+
|
|
89
96
|
## Response Style
|
|
90
97
|
|
|
91
98
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -86,6 +86,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
86
86
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
87
87
|
- Circuit breakers for unhealthy dependencies.
|
|
88
88
|
|
|
89
|
+
## Workflow
|
|
90
|
+
|
|
91
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
92
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
93
|
+
they set up a research/plan gate before implementation. Let the user
|
|
94
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
95
|
+
|
|
89
96
|
## Response Style
|
|
90
97
|
|
|
91
98
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -86,6 +86,13 @@ Before writing any code, stop at the first step that holds:
|
|
|
86
86
|
- Retries use exponential backoff with jitter. Only retry idempotent operations.
|
|
87
87
|
- Circuit breakers for unhealthy dependencies.
|
|
88
88
|
|
|
89
|
+
## Workflow
|
|
90
|
+
|
|
91
|
+
Starting a project from scratch, or adding a non-trivial feature to an
|
|
92
|
+
existing codebase? Mention `/asc-new-project` or `/asc-add-feature` —
|
|
93
|
+
they set up a research/plan gate before implementation. Let the user
|
|
94
|
+
decide whether to invoke it. Skip this for trivial edits.
|
|
95
|
+
|
|
89
96
|
## Response Style
|
|
90
97
|
|
|
91
98
|
Lead with what the developer needs to act: the command, file path, code change, or decision point. Format: direct statement, then evidence. Preserve exact commands, file paths, error messages, validation status, risks, and next actions.
|
|
@@ -1,16 +1,11 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: asc
|
|
3
|
-
description: "Universal AI coding rules. Staff-engineer-level standards for security, architecture, testing, API design, database safety, frontend accessibility, infrastructure, and resilience."
|
|
4
|
-
homepage: https://github.com/fatidaprilian/Agentic-Senior-Core
|
|
5
|
-
license: MIT
|
|
6
|
-
---
|
|
7
|
-
|
|
8
1
|
# Agentic Senior Core
|
|
9
2
|
|
|
10
3
|
Universal AI coding rules. Write code like a staff engineer.
|
|
11
4
|
|
|
12
5
|
## Available Commands
|
|
13
6
|
|
|
7
|
+
- `/asc-new-project` -- Greenfield workflow (Define -> Spec -> Implement -> Validate)
|
|
8
|
+
- `/asc-add-feature` -- Brownfield workflow (Research -> Plan -> Implement)
|
|
14
9
|
- `/asc-refactor` -- Structured refactoring workflow with pre-checks and validation
|
|
15
10
|
- `/asc-review` -- Production-risk code review with severity-ordered findings
|
|
16
11
|
- `/asc-audit` -- Security and architecture audit
|
|
@@ -24,8 +19,9 @@ On plugin-tier hosts (Claude Code, Codex CLI, Copilot CLI), a PostToolUse hook f
|
|
|
24
19
|
- New dependencies against stdlib duplicates (decision ladder step 3)
|
|
25
20
|
- LOC delta > 30 lines on edits (step 5)
|
|
26
21
|
- New files > 50 lines (steps 1–2)
|
|
22
|
+
- Workflow gate violations: source/config edits during research or plan phases
|
|
27
23
|
|
|
28
|
-
Violations inject a nudge referencing the specific ladder step. The hook is silent when no issues are found.
|
|
24
|
+
Violations inject a nudge referencing the specific ladder step or workflow phase. The hook is silent when no issues are found. All enforcement is advisory — nudges, not hard blocks.
|
|
29
25
|
|
|
30
26
|
## What It Does
|
|
31
27
|
|
|
@@ -0,0 +1,37 @@
|
|
|
1
|
+
# ASC Adapter
|
|
2
|
+
|
|
3
|
+
Detect installed AI coding hosts and generate adapter files for the current project.
|
|
4
|
+
|
|
5
|
+
## When to use
|
|
6
|
+
|
|
7
|
+
Run this when setting up a new project or when a developer wants ASC rules active across all their AI coding tools.
|
|
8
|
+
|
|
9
|
+
## Steps
|
|
10
|
+
|
|
11
|
+
1. Run `asc status` to detect which AI coding hosts are installed on this system.
|
|
12
|
+
2. Check which adapter files already exist in the current project directory.
|
|
13
|
+
3. For any detected host that is missing an adapter, run `asc adapter --<host>` to generate it.
|
|
14
|
+
4. Use `asc adapter --all` to generate adapters for all supported hosts at once.
|
|
15
|
+
|
|
16
|
+
## Supported hosts
|
|
17
|
+
|
|
18
|
+
Plugin hosts (always-on, no adapter needed): Claude Code, Codex CLI, Gemini CLI, Antigravity CLI, Copilot CLI, Devin CLI, Hermes, OpenCode, OpenClaw.
|
|
19
|
+
|
|
20
|
+
Adapter hosts (one file per project): Cursor, Devin Desktop, Cline, GitHub Copilot, Kiro, Continue, Zed, Aider, Kilo Code, Roo Code, OpenHands.
|
|
21
|
+
|
|
22
|
+
## Commands
|
|
23
|
+
|
|
24
|
+
```bash
|
|
25
|
+
asc status # Show detected hosts
|
|
26
|
+
asc adapter --all # Generate all adapters
|
|
27
|
+
asc adapter --cursor # Generate for specific host
|
|
28
|
+
asc uninstall # Remove all ASC adapter files
|
|
29
|
+
asc uninstall --dry-run # Preview what would be removed
|
|
30
|
+
```
|
|
31
|
+
|
|
32
|
+
## Notes
|
|
33
|
+
|
|
34
|
+
- Adapter files contain the ASC universal coding rules, compressed to fit within each host's size limits.
|
|
35
|
+
- Cursor uses `.mdc` format with `alwaysApply: true` frontmatter.
|
|
36
|
+
- Windsurf is now Devin Desktop. Use `--devin` for the preferred path, `--windsurf` for legacy.
|
|
37
|
+
- Zed also reads `AGENTS.md` natively, so the adapter is optional.
|
|
@@ -0,0 +1,46 @@
|
|
|
1
|
+
# Add Feature Workflow
|
|
2
|
+
|
|
3
|
+
Structured brownfield workflow. Adapted from QRSPI to prevent context rot and ensure alignment before building.
|
|
4
|
+
|
|
5
|
+
Grounded in: RPI (Dex Horthy, HumanLayer 2025) with corrections from QRSPI 8-stage evolution (Coding Agents Conference, March 2026). Plan-reading illusion fix and instruction budget constraint applied. Stages 2/5/7 adapted; stages 1/3/4/6/8 skipped as too heavyweight for individual-developer workflow.
|
|
6
|
+
|
|
7
|
+
## Gate Mechanism
|
|
8
|
+
|
|
9
|
+
This workflow nudges the agent to stop at each phase boundary, same enforcement tier as the existing decision ladder — not a hard block. Bypasses are logged to the debt ledger.
|
|
10
|
+
|
|
11
|
+
**Known limitation:** Bypass-to-debt-ledger logging is self-reported by the agent, not enforced by the hook. The PostToolUse hook has no MCP access — it nudges the agent to log, but cannot write the debt entry itself.
|
|
12
|
+
|
|
13
|
+
To track phase, write to `workflow-gate.json` via the `state_write` MCP tool.
|
|
14
|
+
Format:
|
|
15
|
+
```json
|
|
16
|
+
{
|
|
17
|
+
"workflow": "asc-add-feature",
|
|
18
|
+
"phase": "<current_phase>",
|
|
19
|
+
"updatedAt": "<ISO-timestamp>"
|
|
20
|
+
}
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
## Phase 1: Research (No Code Changes)
|
|
24
|
+
|
|
25
|
+
1. Write `workflow-gate.json` with phase `research`.
|
|
26
|
+
2. Map existing code: patterns, utilities, dependencies already in use.
|
|
27
|
+
3. Identify what must NOT be rebuilt (e.g., existing validation helpers).
|
|
28
|
+
4. Output a factual research summary.
|
|
29
|
+
5. **STOP and wait for user approval.** Do not plan or implement.
|
|
30
|
+
|
|
31
|
+
## Phase 2: Plan
|
|
32
|
+
|
|
33
|
+
1. On approval of Phase 1, update `workflow-gate.json` phase to `plan`.
|
|
34
|
+
2. Create a numbered, step-by-step implementation plan with specific files, functions, and line references.
|
|
35
|
+
3. Include a "Don't Build" list from the research phase.
|
|
36
|
+
4. **Callout: Plan-Reading Illusion.** Ask the user to explicitly verify the plan against the codebase, not just skim it.
|
|
37
|
+
5. Output the plan.
|
|
38
|
+
6. **STOP and wait for user approval.** Do not implement.
|
|
39
|
+
|
|
40
|
+
## Phase 3: Implement
|
|
41
|
+
|
|
42
|
+
1. On approval of Phase 2, update `workflow-gate.json` phase to `implement`.
|
|
43
|
+
2. Recommend a fresh context (intentional compaction) if the context window is getting full.
|
|
44
|
+
3. Execute the approved plan.
|
|
45
|
+
4. Validate: tests pass, no duplicate code introduced, plan items checked off.
|
|
46
|
+
5. On completion, clear the state in `workflow-gate.json` by overwriting it with `{}`.
|
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
Security and architecture audit. Deeper than review, focused on finding vulnerabilities and structural anti-patterns.
|
|
4
4
|
|
|
5
|
+
Grounded in: OWASP Top 10 (2021), OWASP ASVS v4, CVSS vulnerability report structure, CWE classification.
|
|
6
|
+
|
|
5
7
|
## Audit Scope
|
|
6
8
|
|
|
7
9
|
1. **Trust boundaries**: Every point where external input enters the system. Validate that inputs are sanitized, normalized, and rejected when invalid.
|
|
@@ -1,14 +1,9 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: asc-debt
|
|
3
|
-
description: "Track deferred enforcement violations. Log, list, resolve, and summarize shortcuts deferred during ASC ladder enforcement."
|
|
4
|
-
homepage: https://github.com/fatidaprilian/Agentic-Senior-Core
|
|
5
|
-
license: MIT
|
|
6
|
-
---
|
|
7
|
-
|
|
8
1
|
# Debt Ledger
|
|
9
2
|
|
|
10
3
|
Track deferred enforcement violations. When an ASC ladder nudge fires and the shortcut is accepted rather than fixed, log it here for later resolution.
|
|
11
4
|
|
|
5
|
+
Grounded in: Cunningham's technical debt metaphor (1992). Entry format (ladder step + violation + status lifecycle) is an ASC-specific implementation.
|
|
6
|
+
|
|
12
7
|
## Storage
|
|
13
8
|
|
|
14
9
|
Entries persist in `.agent-context/state/debt-ledger.json` via MCP `state_read`/`state_write`. The ledger is a JSON array of entry objects.
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
# New Project Workflow
|
|
2
|
+
|
|
3
|
+
Structured greenfield workflow. Prevents building before alignment on what to build.
|
|
4
|
+
|
|
5
|
+
Grounded in: Spec-Driven Development (SDD) with scaffolding-spec approach. Specs guide implementation, then the code becomes the source of truth — specs are not maintained as living documents unless the team explicitly opts in.
|
|
6
|
+
|
|
7
|
+
## Gate Mechanism
|
|
8
|
+
|
|
9
|
+
This workflow nudges the agent to stop at each phase boundary, same enforcement tier as the existing decision ladder — not a hard block. Bypasses are logged to the debt ledger.
|
|
10
|
+
|
|
11
|
+
**Known limitation:** Bypass-to-debt-ledger logging is self-reported by the agent, not enforced by the hook. The PostToolUse hook has no MCP access — it nudges the agent to log, but cannot write the debt entry itself.
|
|
12
|
+
|
|
13
|
+
To track phase, write to `workflow-gate.json` via the `state_write` MCP tool.
|
|
14
|
+
Format:
|
|
15
|
+
```json
|
|
16
|
+
{
|
|
17
|
+
"workflow": "asc-new-project",
|
|
18
|
+
"phase": "<current_phase>",
|
|
19
|
+
"updatedAt": "<ISO-timestamp>"
|
|
20
|
+
}
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
## Phase 1: Define (No Code)
|
|
24
|
+
|
|
25
|
+
1. Write `workflow-gate.json` with phase `research`.
|
|
26
|
+
2. Clarify with the user: tech stack, target platform, constraints, non-goals.
|
|
27
|
+
3. Propose a directory structure and module boundaries.
|
|
28
|
+
4. Output a project brief summarizing decisions.
|
|
29
|
+
5. **STOP and wait for user approval.** Do not write specs or code.
|
|
30
|
+
|
|
31
|
+
## Phase 2: Spec (No Implementation Code)
|
|
32
|
+
|
|
33
|
+
1. On approval of Phase 1, update `workflow-gate.json` phase to `plan`.
|
|
34
|
+
2. Write per-feature specs with acceptance criteria and edge cases.
|
|
35
|
+
3. Specs are scaffolding — they guide the build, then the code is the source of truth.
|
|
36
|
+
4. Output specs for review.
|
|
37
|
+
5. **STOP and wait for user approval.** Do not implement.
|
|
38
|
+
|
|
39
|
+
## Phase 3: Implement
|
|
40
|
+
|
|
41
|
+
1. On approval of Phase 2, update `workflow-gate.json` phase to `implement`.
|
|
42
|
+
2. Build against the approved specs. Apply the ASC decision ladder on every file.
|
|
43
|
+
3. Run the decision ladder: does this need to exist? Does stdlib cover it? One function or full module?
|
|
44
|
+
|
|
45
|
+
## Phase 4: Validate
|
|
46
|
+
|
|
47
|
+
1. Run tests. Check each spec's acceptance criteria.
|
|
48
|
+
2. Confirm nothing was over-built beyond what the specs required.
|
|
49
|
+
3. On completion, clear the state in `workflow-gate.json` by overwriting it with `{}`.
|
|
@@ -0,0 +1,56 @@
|
|
|
1
|
+
# Refactor Skill
|
|
2
|
+
|
|
3
|
+
Structured refactoring workflow. Preserves existing behavior while improving structure.
|
|
4
|
+
|
|
5
|
+
Grounded in: Fowler's Refactoring (read-understand-smallest scope-preserve behavior), Rule of Three (abstraction threshold), YAGNI principle (XP/Kent Beck). Empirical evidence: agents dominate low-level refactors (rename, extract, type changes) but struggle with multi-file architectural changes (arXiv, 15k+ instance study).
|
|
6
|
+
|
|
7
|
+
## YAGNI Scan (Before Any Restructuring)
|
|
8
|
+
|
|
9
|
+
Before changing structure, scan for speculative code to remove:
|
|
10
|
+
- "Just in case" logic, unused feature flags, dead branches behind config toggles.
|
|
11
|
+
- Abstractions wrapping a single implementation with no second consumer.
|
|
12
|
+
- Premature extension points (plugin hooks, strategy patterns) with one concrete path.
|
|
13
|
+
|
|
14
|
+
Removing speculative code is the highest-value, lowest-risk refactor type.
|
|
15
|
+
|
|
16
|
+
## Classify Before Proceeding
|
|
17
|
+
|
|
18
|
+
After reading the target code, classify the refactor:
|
|
19
|
+
|
|
20
|
+
- **Low-level** (rename, extract method, type safety, dead code removal): proceed after explaining the change. No approval gate needed.
|
|
21
|
+
- **High-level** (move module, change architecture, redesign abstractions, multi-file structural changes): output a plan with specific files and rationale, then **stop and wait for user approval** before implementing.
|
|
22
|
+
|
|
23
|
+
**Known limitation:** The high-level refactor gate is a skill-text instruction — the agent self-classifies and self-stops. Unlike `/asc-add-feature` and `/asc-new-project`, this gate is not backed by the PostToolUse hook or workflow-gate.json. If the agent bypasses the gate, there is no automated nudge.
|
|
24
|
+
|
|
25
|
+
## Before Editing
|
|
26
|
+
|
|
27
|
+
1. Read the target code and understand existing patterns.
|
|
28
|
+
2. Identify the smallest relevant scope for the refactor.
|
|
29
|
+
3. Classify the refactor (see above). If high-level, stop and present a plan.
|
|
30
|
+
4. If the change touches UI, check accessibility and responsive behavior.
|
|
31
|
+
5. If the change touches dependencies, verify current official docs.
|
|
32
|
+
|
|
33
|
+
## Refactor Rules
|
|
34
|
+
|
|
35
|
+
- Improve clarity, boundaries, naming, validation, error handling, and tests.
|
|
36
|
+
- Prioritize maintainability over compressed one-liners.
|
|
37
|
+
- Keep the main flow traceable. Use early returns where they reduce nesting.
|
|
38
|
+
- Introduce abstractions only when the repeated pattern is real and visible.
|
|
39
|
+
- Split large files when the split makes the flow easier to understand.
|
|
40
|
+
- Remove code that does not carry behavior, safety, clarity, maintainability, or test value.
|
|
41
|
+
- Prefer the shorter implementation only when it keeps the same guarantees.
|
|
42
|
+
- Run a final simplification pass before completion.
|
|
43
|
+
- Update tests and docs whenever behavior contracts, public APIs, data shape, or UI contracts change.
|
|
44
|
+
|
|
45
|
+
## For Every Change, Explain
|
|
46
|
+
|
|
47
|
+
- What risk or friction existed.
|
|
48
|
+
- What changed.
|
|
49
|
+
- Why the new shape is safer or easier to maintain.
|
|
50
|
+
|
|
51
|
+
## Validation
|
|
52
|
+
|
|
53
|
+
- Existing behavior is preserved unless the user approved a change.
|
|
54
|
+
- Edge cases, empty states, error paths, and rollback paths are handled.
|
|
55
|
+
- Public contracts remain stable or are versioned.
|
|
56
|
+
- Tests pass.
|
|
@@ -1,14 +1,9 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: asc-reference
|
|
3
|
-
description: "Domain-specific coding rules for testing, API design, database queries, frontend components, infrastructure configs, and service resilience."
|
|
4
|
-
homepage: https://github.com/fatidaprilian/Agentic-Senior-Core
|
|
5
|
-
license: MIT
|
|
6
|
-
---
|
|
7
|
-
|
|
8
1
|
# ASC Domain Reference
|
|
9
2
|
|
|
10
3
|
Domain-specific coding rules for testing, API design, database queries, frontend components, infrastructure configs, and service resilience. Load this skill when working on any of these domains.
|
|
11
4
|
|
|
5
|
+
Grounded in: WCAG 2.2 AA (accessibility), Fowler's Money Pattern (monetary types), Nygard's Release It! (resilience patterns), AWS Well-Architected Reliability Pillar (backoff/circuit breakers), OWASP Secure Coding Practices (input validation, logging).
|
|
6
|
+
|
|
12
7
|
## Testing
|
|
13
8
|
|
|
14
9
|
- Write tests for business logic and boundary failures, not implementation details.
|
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
Production-risk code review. Prioritize findings by severity.
|
|
4
4
|
|
|
5
|
+
Grounded in: OWASP Risk Rating Methodology, Google Engineering Practices (code review guide), Clean Architecture (transport vs business layer separation).
|
|
6
|
+
|
|
5
7
|
## Before Reviewing
|
|
6
8
|
|
|
7
9
|
1. Read the changed files and understand the scope.
|