Compare commits
8 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| a416413c7d | |||
| e8c6c5c917 | |||
| 4de1f65c10 | |||
| 6902c37ced | |||
| bbabd2dc0a | |||
| 99df4fe4e2 | |||
| 8527df24b3 | |||
| 4a58aa1657 |
@@ -53,7 +53,7 @@ src/
|
||||
security.ts # Layer 3: regex-based threat pattern scanning (no STRIDE analysis yet)
|
||||
quality.ts # Layer 4: regex-based code quality checks (no multi-persona review yet)
|
||||
index.ts # Public API exports
|
||||
version.ts # VERSION = "0.6.0"
|
||||
version.ts # VERSION = "0.7.0"
|
||||
templates/ # Template files (config.json, DECISIONS.md, specification.md)
|
||||
```
|
||||
|
||||
@@ -82,10 +82,10 @@ templates/ # Template files (config.json, DECISIONS.md, specification.md
|
||||
## Pipeline Flow
|
||||
|
||||
```
|
||||
SPECIFY → CLARIFY → RESEARCH → PLAN → EXECUTE → VERIFY → COMPLETE
|
||||
SPECIFY → CLARIFY → RESEARCH → PLAN → EXECUTE → TEST → VERIFY → COMPLETE
|
||||
```
|
||||
|
||||
Each stage is executed by `OrchestratorAgent.executeStage()`. The orchestrator delegates intelligent stages (research, plan, execute, verify) to specialized agents via `context.backend` when available, falling back to mechanical execution when no backend is configured. Mechanical stages (specify, clarify, complete) are always handled by the orchestrator directly.
|
||||
Each stage is executed by `OrchestratorAgent.executeStage()`. The orchestrator delegates intelligent stages (research, plan, execute, test, verify) to specialized agents via `context.backend` when available, falling back to mechanical execution when no backend is configured. Mechanical stages (specify, clarify, complete) are always handled by the orchestrator directly.
|
||||
|
||||
## Intelligence Backend Architecture
|
||||
|
||||
@@ -191,7 +191,7 @@ IntelligenceBackend (unified interface)
|
||||
|
||||
## Current State
|
||||
|
||||
- **v0.6.0**: Backends module (OllamaLocal, OllamaCloud, Opencode), learnship references removed, verification layers migrated from .planning/ to .ciagent/
|
||||
- **v0.7.0**: Backends module (OllamaLocal, OllamaCloud, Opencode), learnship references removed, verification layers migrated from .planning/ to .ciagent/
|
||||
- **New modules**: commit-parser (`---ci---` YAML block extraction/parsing), commit-builder (structured commit message generation), git-context (project state reconstruction from git log + branches), git-branch (phase/milestone branch lifecycle), ciagent-files (`.ciagent/` long-lived reference file management)
|
||||
- **Commit schema**: Every CIAgent-generated commit contains a `---ci---` YAML block with phase, milestone, status, decisions, escalations, requirements, lessons, and compound metadata
|
||||
- **Branch strategy**: `phase/NN-slug` and `milestone/vX.X-slug` branches encode project structure; merged = complete, active = in progress
|
||||
|
||||
@@ -211,9 +211,9 @@ CIAgent uses `.ciagent/config.json` for project configuration:
|
||||
### Pipeline
|
||||
|
||||
```
|
||||
SPECIFY → CLARIFY → RESEARCH → PLAN → EXECUTE → VERIFY → COMPLETE
|
||||
↕ ↕ ↕ ↕
|
||||
(questions) (auto-decide) (auto-run) (auto-verify)
|
||||
SPECIFY → CLARIFY → RESEARCH → PLAN → EXECUTE → TEST → VERIFY → COMPLETE
|
||||
↕ ↕ ↕ ↕ ↕
|
||||
(questions) (auto-decide) (auto-run) (auto-test) (auto-verify)
|
||||
```
|
||||
|
||||
### Git-Native Core Modules
|
||||
@@ -235,7 +235,7 @@ Every autonomous decision is classified by confidence:
|
||||
|
||||
Decisions are committed to git as `decision` type commits. The audit trail is `git log --grep="decisions:"`.
|
||||
|
||||
### 18 Agents
|
||||
### 19 Agents
|
||||
|
||||
| Agent | Role | CIAgent Modification |
|
||||
|-------|------|----------------|
|
||||
@@ -244,17 +244,18 @@ Decisions are committed to git as `decision` type commits. The audit trail is `g
|
||||
| executor | Task execution | Never pauses for checkpoints |
|
||||
| verifier | Output verification | Generates automated tests, not human UAT |
|
||||
| researcher | Domain research | Logs assumptions, never flags for human |
|
||||
| tester | Integration/e2e tests | Detects and runs existing test files, never writes tests |
|
||||
| challenger | Plan stress-testing | Binding verdicts, only escalates <0.60 |
|
||||
| security-auditor | Security audit | Auto-dispositions threats |
|
||||
| debugger | Bug fixing | Auto-fixes when confidence > threshold |
|
||||
| Others | Various | Retained from Learnship |
|
||||
| Others | Various | Delegates to active intelligence backend |
|
||||
|
||||
### Verification Layers
|
||||
|
||||
1. **Structural**: File existence, import/export wiring, no stubs
|
||||
2. **Behavioral**: Generated automated tests for must-haves
|
||||
3. **Security**: STRIDE analysis with auto-disposition
|
||||
4. **Code Quality**: Multi-persona review with P0 auto-fix
|
||||
2. **Behavioral**: Test infrastructure and requirement traceability (partially implemented — static analysis, no test generation yet)
|
||||
3. **Security**: Regex-based threat pattern scanning with auto-disposition (partially implemented — no STRIDE analysis yet)
|
||||
4. **Code Quality**: Regex-based code quality checks (partially implemented — no multi-persona review yet)
|
||||
|
||||
## Specification Format
|
||||
|
||||
@@ -292,9 +293,9 @@ Each escalation is committed as an `escalation` type commit. Resolved escalation
|
||||
|
||||
## Current Limitations
|
||||
|
||||
- **Agent implementations are stubs**: All 18 agents return success immediately. Real LLM-based agent implementations are needed for research, planning, execution, and verification.
|
||||
- **Agent implementations**: 5 core agents have intrinsic logic (planner, executor, verifier, researcher, tester); 13 agents delegate to backends. Full LLM-powered agent behavior requires an intelligence backend.
|
||||
- **Package not published to npm**: Install from source only until a publishing pipeline is configured.
|
||||
- **Behavioral/Security/Quality verification layers**: Structural verification is fully implemented; behavioral, security, and quality layers are partially stubbed.
|
||||
- **Behavioral/Security/Quality verification layers**: Partially implemented — structural verification is complete; behavioral does static analysis; security does regex-based threat scanning; quality does regex-based code quality checks.
|
||||
|
||||
## Differences from Learnship
|
||||
|
||||
|
||||
+1
-1
@@ -1 +1 @@
|
||||
0.5.0
|
||||
0.7.0
|
||||
Generated
+5
-5
@@ -1,12 +1,12 @@
|
||||
{
|
||||
"name": "@continuous-intelligence/ci",
|
||||
"version": "0.4.0",
|
||||
"name": "@continuous-intelligence/ciagent",
|
||||
"version": "0.5.0",
|
||||
"lockfileVersion": 3,
|
||||
"requires": true,
|
||||
"packages": {
|
||||
"": {
|
||||
"name": "@continuous-intelligence/ci",
|
||||
"version": "0.4.0",
|
||||
"name": "@continuous-intelligence/ciagent",
|
||||
"version": "0.5.0",
|
||||
"hasInstallScript": true,
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
@@ -14,7 +14,7 @@
|
||||
"zod": "^3.23.0"
|
||||
},
|
||||
"bin": {
|
||||
"ci": "dist/cli/index.js"
|
||||
"ciagent": "dist/cli/index.js"
|
||||
},
|
||||
"devDependencies": {
|
||||
"@types/jest": "^29.5.0",
|
||||
|
||||
+17
-5
@@ -1,11 +1,11 @@
|
||||
{
|
||||
"name": "@continuous-intelligence/ci",
|
||||
"version": "0.5.0",
|
||||
"name": "@continuous-intelligence/ciagent",
|
||||
"version": "0.7.0",
|
||||
"description": "Fully autonomous AI-driven software engineering harness - Continuous Intelligence",
|
||||
"main": "dist/index.js",
|
||||
"types": "dist/index.d.ts",
|
||||
"bin": {
|
||||
"ci": "./dist/cli/index.js"
|
||||
"ciagent": "./dist/cli/index.js"
|
||||
},
|
||||
"files": [
|
||||
"dist/",
|
||||
@@ -19,14 +19,26 @@
|
||||
"dev": "ts-node src/cli.ts",
|
||||
"typecheck": "tsc --noEmit",
|
||||
"test": "jest",
|
||||
"prepublishOnly": "npm run build",
|
||||
"prepublishOnly": "npm run build && npm test",
|
||||
"install-opencode": "node scripts/postinstall.js"
|
||||
},
|
||||
"keywords": ["ci", "autonomous", "ai", "software-engineering", "agent", "multi-project"],
|
||||
"keywords": ["ciagent", "autonomous", "ai", "software-engineering", "agent", "multi-project"],
|
||||
"license": "MIT",
|
||||
"engines": {
|
||||
"node": ">=18.0.0"
|
||||
},
|
||||
"publishConfig": {
|
||||
"registry": "https://registry.npmjs.org/",
|
||||
"access": "public"
|
||||
},
|
||||
"repository": {
|
||||
"type": "git",
|
||||
"url": "https://git.cloudinit.dev/continuous-intelligence/ciagent.git"
|
||||
},
|
||||
"homepage": "https://git.cloudinit.dev/continuous-intelligence/ciagent",
|
||||
"bugs": {
|
||||
"url": "https://git.cloudinit.dev/continuous-intelligence/ciagent/issues"
|
||||
},
|
||||
"dependencies": {
|
||||
"commander": "^12.1.0",
|
||||
"zod": "^3.23.0"
|
||||
|
||||
@@ -2,7 +2,7 @@ import { BaseAgent, AgentContext, AgentResult } from "./base.js";
|
||||
|
||||
export class DocWriterAgent extends BaseAgent {
|
||||
readonly name = "doc-writer";
|
||||
readonly description = "Autonomous documentation writer. No behavioral changes from Learnship.";
|
||||
readonly description = "Autonomous documentation writer.";
|
||||
readonly workflow = "execute";
|
||||
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
|
||||
@@ -0,0 +1,79 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { ExecutorAgent } from "../agents/executor.js";
|
||||
import { AgentContext } from "../agents/base.js";
|
||||
import { IntelligenceBackend, BackendRequest, BackendResult } from "../backends/types.js";
|
||||
import { emptyTokenUsage } from "../backends/types.js";
|
||||
|
||||
class MockBackend implements IntelligenceBackend {
|
||||
readonly name = "mock";
|
||||
readonly type = "llm" as const;
|
||||
async isAvailable(): Promise<boolean> { return true; }
|
||||
async execute(request: BackendRequest): Promise<BackendResult> {
|
||||
return {
|
||||
success: true,
|
||||
output: `Mock backend executed: ${request.task.slice(0, 50)}`,
|
||||
artifacts: [],
|
||||
decisions: [],
|
||||
escalations: [],
|
||||
usage: emptyTokenUsage(),
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-executor-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
function makeContext(dir: string, backend?: IntelligenceBackend): AgentContext {
|
||||
return {
|
||||
project_path: dir,
|
||||
phase: 1,
|
||||
stage: "execute",
|
||||
specification: "Build a REST API for task management",
|
||||
config_path: path.join(dir, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
}
|
||||
|
||||
describe("ExecutorAgent", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = createTempDir();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
it("returns honest failure without backend", async () => {
|
||||
const executor = new ExecutorAgent();
|
||||
const result = await executor.execute(makeContext(dir));
|
||||
expect(result.success).toBe(false);
|
||||
expect(result.error).toContain("intelligence backend");
|
||||
});
|
||||
|
||||
it("delegates to backend when available", async () => {
|
||||
const mockBackend = new MockBackend();
|
||||
const executor = new ExecutorAgent();
|
||||
const result = await executor.execute(makeContext(dir, mockBackend));
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("Mock backend executed");
|
||||
});
|
||||
|
||||
it("has correct agent name", () => {
|
||||
const executor = new ExecutorAgent();
|
||||
expect(executor.name).toBe("executor");
|
||||
});
|
||||
|
||||
it("has correct workflow", () => {
|
||||
const executor = new ExecutorAgent();
|
||||
expect(executor.workflow).toBe("execute");
|
||||
});
|
||||
});
|
||||
+163
-7
@@ -1,4 +1,21 @@
|
||||
import { BaseAgent, AgentContext, AgentResult } from "./base.js";
|
||||
import { execSync } from "node:child_process";
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
|
||||
export interface ExecutorResult {
|
||||
success: boolean;
|
||||
tasksExecuted: number;
|
||||
tasksCommitted: number;
|
||||
testsPassing: boolean;
|
||||
mustHavesChecked: { name: string; passed: boolean }[];
|
||||
error?: string;
|
||||
}
|
||||
|
||||
interface MustHaveItem {
|
||||
name: string;
|
||||
passed: boolean;
|
||||
}
|
||||
|
||||
export class ExecutorAgent extends BaseAgent {
|
||||
readonly name = "executor";
|
||||
@@ -8,21 +25,160 @@ export class ExecutorAgent extends BaseAgent {
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
const start = Date.now();
|
||||
this.log("Executing tasks...");
|
||||
|
||||
if (context.backend) {
|
||||
const result = await this.executeViaBackend(
|
||||
context,
|
||||
`Execute implementation for stage ${context.stage}, phase ${context.phase}. Specification: ${context.specification}`
|
||||
);
|
||||
return { ...result, duration_ms: Date.now() - start };
|
||||
const taskPrompt = await this.buildBackendTaskPrompt(context);
|
||||
const backendResult = await this.executeViaBackend(context, taskPrompt);
|
||||
|
||||
const verification = await this.verifyExecution(context);
|
||||
|
||||
return {
|
||||
...backendResult,
|
||||
output: `${backendResult.output}\nVerification: tests=${verification.testsPassing ? "passing" : "failing"}, must-haves checked=${verification.mustHavesChecked.length}`,
|
||||
duration_ms: Date.now() - start,
|
||||
};
|
||||
}
|
||||
|
||||
return {
|
||||
success: false,
|
||||
output: "Execution requires an intelligence backend. Configure one with: ci init --backend",
|
||||
output: "Executor requires intelligence backend for code implementation",
|
||||
artifacts_created: [],
|
||||
decisions: 0,
|
||||
escalations: 0,
|
||||
duration_ms: Date.now() - start,
|
||||
error: "No intelligence backend available",
|
||||
error: "Executor requires intelligence backend for code implementation",
|
||||
};
|
||||
}
|
||||
|
||||
private async buildBackendTaskPrompt(context: AgentContext): Promise<string> {
|
||||
const parts: string[] = [
|
||||
`Execute implementation for stage ${context.stage}, phase ${context.phase}.`,
|
||||
"",
|
||||
"## Specification",
|
||||
context.specification || "No specification provided",
|
||||
];
|
||||
|
||||
const planContent = this.readPlanFile(context);
|
||||
if (planContent) {
|
||||
parts.push("", "## Plan", planContent);
|
||||
}
|
||||
|
||||
const ciDir = path.join(context.project_path, ".ciagent");
|
||||
const roadmapPath = path.join(ciDir, "ROADMAP.md");
|
||||
const archPath = path.join(ciDir, "ARCHITECTURE.md");
|
||||
|
||||
if (fs.existsSync(roadmapPath)) {
|
||||
try {
|
||||
const roadmap = fs.readFileSync(roadmapPath, "utf-8");
|
||||
parts.push("", "## Roadmap Context", roadmap.slice(0, 2000));
|
||||
} catch {}
|
||||
}
|
||||
|
||||
if (fs.existsSync(archPath)) {
|
||||
try {
|
||||
const arch = fs.readFileSync(archPath, "utf-8");
|
||||
parts.push("", "## Architecture Boundaries", arch.slice(0, 2000));
|
||||
} catch {}
|
||||
}
|
||||
|
||||
parts.push("", "## Execution Rules");
|
||||
parts.push("- Execute one task at a time");
|
||||
parts.push("- Commit after each task with ---ci--- block");
|
||||
parts.push("- Never pause for checkpoints");
|
||||
parts.push("- Create automated verification for traditionally human tasks");
|
||||
|
||||
return parts.join("\n");
|
||||
}
|
||||
|
||||
private readPlanFile(context: AgentContext): string | null {
|
||||
const planPath = path.join(context.project_path, ".ciagent", "PLAN.md");
|
||||
try {
|
||||
if (fs.existsSync(planPath)) {
|
||||
return fs.readFileSync(planPath, "utf-8");
|
||||
}
|
||||
} catch {}
|
||||
return null;
|
||||
}
|
||||
|
||||
private async verifyExecution(context: AgentContext): Promise<ExecutorResult> {
|
||||
const mustHavesChecked: MustHaveItem[] = this.checkMustHaves(context);
|
||||
let testsPassing = false;
|
||||
let tasksExecuted = 0;
|
||||
let tasksCommitted = 0;
|
||||
|
||||
try {
|
||||
const logOutput = execSync("git log --max-count=20 --oneline", {
|
||||
cwd: context.project_path,
|
||||
encoding: "utf-8",
|
||||
stdio: ["pipe", "pipe", "pipe"],
|
||||
}).trim();
|
||||
const commitLines = logOutput.split("\n").filter(Boolean);
|
||||
tasksCommitted = commitLines.filter((l) => /feat|fix|test/.test(l)).length;
|
||||
tasksExecuted = tasksCommitted;
|
||||
} catch {}
|
||||
|
||||
try {
|
||||
execSync("npm test", {
|
||||
cwd: context.project_path,
|
||||
encoding: "utf-8",
|
||||
stdio: ["pipe", "pipe", "pipe"],
|
||||
timeout: 120000,
|
||||
});
|
||||
testsPassing = true;
|
||||
} catch {
|
||||
testsPassing = false;
|
||||
}
|
||||
|
||||
return {
|
||||
success: mustHavesChecked.every((m) => m.passed) && testsPassing,
|
||||
tasksExecuted,
|
||||
tasksCommitted,
|
||||
testsPassing,
|
||||
mustHavesChecked,
|
||||
};
|
||||
}
|
||||
|
||||
private checkMustHaves(context: AgentContext): MustHaveItem[] {
|
||||
const planPath = path.join(context.project_path, ".ciagent", "PLAN.md");
|
||||
const results: MustHaveItem[] = [];
|
||||
|
||||
try {
|
||||
if (!fs.existsSync(planPath)) return results;
|
||||
const planContent = fs.readFileSync(planPath, "utf-8");
|
||||
const mustHaveRegex = /-\s*\[x\]\s*(.+)/g;
|
||||
let match;
|
||||
while ((match = mustHaveRegex.exec(planContent)) !== null) {
|
||||
const name = match[1].trim();
|
||||
const passed = this.verifyMustHaveItem(name, context);
|
||||
results.push({ name, passed });
|
||||
}
|
||||
} catch {}
|
||||
|
||||
return results;
|
||||
}
|
||||
|
||||
private verifyMustHaveItem(item: string, context: AgentContext): boolean {
|
||||
const fileMatch = item.match(/(?:exists|created?|present).*?[\s:]+([^\s]+\.(ts|js|json|md))/i);
|
||||
if (fileMatch) {
|
||||
const filePath = path.join(context.project_path, fileMatch[1]);
|
||||
return fs.existsSync(filePath);
|
||||
}
|
||||
|
||||
const testMatch = item.match(/(?:test|tests?)\s+(?:pass|passing)/i);
|
||||
if (testMatch) {
|
||||
try {
|
||||
execSync("npm test", {
|
||||
cwd: context.project_path,
|
||||
encoding: "utf-8",
|
||||
stdio: ["pipe", "pipe", "pipe"],
|
||||
timeout: 120000,
|
||||
});
|
||||
return true;
|
||||
} catch {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
}
|
||||
+7
-4
@@ -1,9 +1,9 @@
|
||||
export { BaseAgent, AgentContext, AgentResult, backendResultToAgentResult } from "./base.js";
|
||||
export { OrchestratorAgent } from "./orchestrator.js";
|
||||
export { PlannerAgent } from "./planner.js";
|
||||
export { ExecutorAgent } from "./executor.js";
|
||||
export { VerifierAgent } from "./verifier.js";
|
||||
export { ResearcherAgent } from "./researcher.js";
|
||||
export { PlannerAgent, PlannerResult } from "./planner.js";
|
||||
export { ExecutorAgent, ExecutorResult } from "./executor.js";
|
||||
export { VerifierAgent, VerifierResult } from "./verifier.js";
|
||||
export { ResearcherAgent, ResearcherResult } from "./researcher.js";
|
||||
export { ChallengerAgent } from "./challenger.js";
|
||||
export { SecurityAuditorAgent } from "./security-auditor.js";
|
||||
export { DebuggerAgent } from "./debugger.js";
|
||||
@@ -17,6 +17,7 @@ export { ProjectResearcherAgent } from "./project-researcher.js";
|
||||
export { ResearchSynthesizerAgent } from "./research-synthesizer.js";
|
||||
export { SolutionWriterAgent } from "./solution-writer.js";
|
||||
export { PhaseResearcherAgent } from "./phase-researcher.js";
|
||||
export { TesterAgent, TesterResult } from "./tester.js";
|
||||
|
||||
import { AgentName } from "../types/config.js";
|
||||
import { BaseAgent as BaseAgentType } from "./base.js";
|
||||
@@ -38,6 +39,7 @@ import { ProjectResearcherAgent } from "./project-researcher.js";
|
||||
import { ResearchSynthesizerAgent } from "./research-synthesizer.js";
|
||||
import { SolutionWriterAgent } from "./solution-writer.js";
|
||||
import { PhaseResearcherAgent } from "./phase-researcher.js";
|
||||
import { TesterAgent } from "./tester.js";
|
||||
|
||||
const agentRegistry: Record<AgentName, () => BaseAgentType> = {
|
||||
orchestrator: () => new OrchestratorAgent(),
|
||||
@@ -58,6 +60,7 @@ const agentRegistry: Record<AgentName, () => BaseAgentType> = {
|
||||
"project-researcher": () => new ProjectResearcherAgent(),
|
||||
"research-synthesizer": () => new ResearchSynthesizerAgent(),
|
||||
"solution-writer": () => new SolutionWriterAgent(),
|
||||
tester: () => new TesterAgent(),
|
||||
};
|
||||
|
||||
export function getAgent(name: AgentName): BaseAgentType {
|
||||
|
||||
+346
-67
@@ -4,9 +4,9 @@ import { ClarifyPhase } from "../core/clarify.js";
|
||||
import { EscalationProtocol, EscalationInput } from "../core/escalation.js";
|
||||
import { GitContext, ProjectState } from "../core/git-context.js";
|
||||
import { GitBranch } from "../core/git-branch.js";
|
||||
import { CiFiles } from "../core/ci-files.js";
|
||||
import { CIAgentFiles } from "../core/ciagent-files.js";
|
||||
import { CommitBuilder } from "../core/commit-builder.js";
|
||||
import { CIConfig, AgentName } from "../types/config.js";
|
||||
import { CIAgentConfig, AgentName } from "../types/config.js";
|
||||
import {
|
||||
PipelineState,
|
||||
PipelineStage,
|
||||
@@ -16,40 +16,44 @@ import {
|
||||
STAGE_ORDER,
|
||||
} from "../types/pipeline.js";
|
||||
import { Specification, parseSpecification } from "../types/specification.js";
|
||||
import { loadConfig, saveConfig, isCIInitialized, initCI } from "../core/config.js";
|
||||
import { loadConfig, saveConfig, isCIAgentInitialized, initCIAgent } from "../core/config.js";
|
||||
import { getAgent } from "./index.js";
|
||||
import { IntelligenceBackend, BackendUnavailableError } from "../backends/types.js";
|
||||
import { execSync } from "node:child_process";
|
||||
|
||||
export interface GitAgentContext extends AgentContext {
|
||||
gitContext: GitContext;
|
||||
gitBranch: GitBranch;
|
||||
ciFiles: CiFiles;
|
||||
ciFiles: CIAgentFiles;
|
||||
milestone: string;
|
||||
}
|
||||
|
||||
export class OrchestratorAgent extends BaseAgent {
|
||||
readonly name: AgentName = "orchestrator";
|
||||
readonly description = "Top-level autonomous controller that coordinates the full CI pipeline";
|
||||
readonly description = "Top-level autonomous controller that coordinates the full CIAgent pipeline";
|
||||
readonly workflow = "run";
|
||||
|
||||
private config: CIConfig;
|
||||
private config: CIAgentConfig;
|
||||
private pipelineState: PipelineState | null = null;
|
||||
private decisionEngine: DecisionEngine | null = null;
|
||||
private escalationProtocol: EscalationProtocol | null = null;
|
||||
private gitContext: GitContext | null = null;
|
||||
private gitBranch: GitBranch | null = null;
|
||||
private ciFiles: CiFiles | null = null;
|
||||
private ciFiles: CIAgentFiles | null = null;
|
||||
private currentMilestone: string;
|
||||
private phaseResults: PhaseResult[] = [];
|
||||
private totalPhases: number = 1;
|
||||
|
||||
private static readonly STAGE_AGENT_MAP: Partial<Record<PipelineStage, AgentName>> = {
|
||||
research: "researcher",
|
||||
plan: "planner",
|
||||
execute: "executor",
|
||||
verify: "verifier",
|
||||
private static readonly STAGE_AGENT_MAP: Partial<Record<PipelineStage, AgentName[]>> = {
|
||||
research: ["researcher"],
|
||||
plan: ["planner"],
|
||||
execute: ["executor", "code-reviewer", "security-auditor"],
|
||||
test: ["tester"],
|
||||
verify: ["verifier"],
|
||||
complete: ["doc-writer"],
|
||||
};
|
||||
|
||||
constructor(config?: CIConfig) {
|
||||
constructor(config?: CIAgentConfig) {
|
||||
super();
|
||||
this.config = config || loadConfig(process.cwd());
|
||||
this.currentMilestone = "v1.0";
|
||||
@@ -57,14 +61,14 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
const startTime = Date.now();
|
||||
this.log("Starting CI Orchestrator pipeline (git-native)");
|
||||
this.log("Starting CIAgent Orchestrator pipeline (git-native)");
|
||||
|
||||
try {
|
||||
this.config = loadConfig(context.project_path);
|
||||
|
||||
this.gitContext = new GitContext(context.project_path);
|
||||
this.gitBranch = new GitBranch(context.project_path);
|
||||
this.ciFiles = new CiFiles(context.project_path);
|
||||
this.ciFiles = new CIAgentFiles(context.project_path);
|
||||
this.ciFiles.ensureCIDir();
|
||||
|
||||
const projectState = this.gitContext.reconstructState();
|
||||
@@ -78,47 +82,66 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
this.pipelineState.current_stage = projectState.currentStage;
|
||||
}
|
||||
|
||||
this.totalPhases = this.deriveTotalPhases();
|
||||
this.log(`Total phases in milestone: ${this.totalPhases}`);
|
||||
|
||||
this.decisionEngine = new DecisionEngine(this.config, context.project_path, this.currentMilestone);
|
||||
this.escalationProtocol = new EscalationProtocol(this.config, context.project_path, this.currentMilestone);
|
||||
|
||||
for (const stage of STAGE_ORDER) {
|
||||
this.log(`Entering stage: ${stage}`);
|
||||
this.pipelineState.current_stage = stage;
|
||||
this.pipelineState.last_updated = new Date().toISOString();
|
||||
while (this.pipelineState.current_phase <= this.totalPhases) {
|
||||
this.log(`Processing phase ${this.pipelineState.current_phase} of ${this.totalPhases}`);
|
||||
|
||||
const result = await this.executeStage(stage, context);
|
||||
for (const stage of STAGE_ORDER) {
|
||||
this.log(`Entering stage: ${stage}`);
|
||||
this.pipelineState.current_stage = stage;
|
||||
this.pipelineState.last_updated = new Date().toISOString();
|
||||
|
||||
if (!result.success && stage !== "complete") {
|
||||
this.pipelineState.errors.push({
|
||||
stage,
|
||||
phase: this.pipelineState.current_phase,
|
||||
message: result.error || "Stage failed",
|
||||
timestamp: new Date().toISOString(),
|
||||
retry_count: 0,
|
||||
resolved: false,
|
||||
});
|
||||
const result = await this.executeStageWithRecovery(stage, context);
|
||||
|
||||
if (stage === "specify" || stage === "clarify") {
|
||||
return {
|
||||
success: false,
|
||||
output: `Pipeline failed at ${stage}: ${result.error}`,
|
||||
artifacts_created: this.phaseResults.reduce(
|
||||
(acc, r) => acc + r.artifacts_created.length,
|
||||
0
|
||||
),
|
||||
decisions: this.phaseResults.reduce(
|
||||
(acc, r) => acc + r.decisions_made,
|
||||
0
|
||||
),
|
||||
escalations: this.phaseResults.reduce(
|
||||
(acc, r) => acc + r.escalations_raised,
|
||||
0
|
||||
),
|
||||
duration_ms: Date.now() - startTime,
|
||||
error: result.error,
|
||||
};
|
||||
this.phaseResults.push(result);
|
||||
this.recordPhaseResult(result);
|
||||
|
||||
if (!result.success && stage !== "complete") {
|
||||
this.pipelineState.errors.push({
|
||||
stage,
|
||||
phase: this.pipelineState.current_phase,
|
||||
message: result.error || "Stage failed",
|
||||
timestamp: new Date().toISOString(),
|
||||
retry_count: 0,
|
||||
resolved: false,
|
||||
});
|
||||
|
||||
if (stage === "specify" || stage === "clarify") {
|
||||
return {
|
||||
success: false,
|
||||
output: `Pipeline failed at ${stage}: ${result.error}`,
|
||||
artifacts_created: this.phaseResults.reduce(
|
||||
(acc, r) => acc + r.artifacts_created.length,
|
||||
0
|
||||
),
|
||||
decisions: this.phaseResults.reduce(
|
||||
(acc, r) => acc + r.decisions_made,
|
||||
0
|
||||
),
|
||||
escalations: this.phaseResults.reduce(
|
||||
(acc, r) => acc + r.escalations_raised,
|
||||
0
|
||||
),
|
||||
duration_ms: Date.now() - startTime,
|
||||
error: result.error,
|
||||
};
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (this.pipelineState.current_phase < this.totalPhases) {
|
||||
this.performPhaseBoundaryCheckpoint(context);
|
||||
this.pipelineState.current_phase++;
|
||||
this.pipelineState.current_stage = "specify";
|
||||
this.log(`Advancing to phase ${this.pipelineState.current_phase}`);
|
||||
} else {
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
const totalDuration = Date.now() - startTime;
|
||||
@@ -151,36 +174,240 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
duration_ms: Date.now() - startTime,
|
||||
error: err instanceof Error ? err.message : String(err),
|
||||
};
|
||||
} finally {
|
||||
this.escalationProtocol?.dispose();
|
||||
}
|
||||
}
|
||||
|
||||
private buildGitAgentContext(context: AgentContext): GitAgentContext {
|
||||
return {
|
||||
...context,
|
||||
gitContext: this.gitContext!,
|
||||
gitBranch: this.gitBranch!,
|
||||
ciFiles: this.ciFiles!,
|
||||
milestone: this.currentMilestone,
|
||||
};
|
||||
}
|
||||
|
||||
private recordPhaseResult(result: PhaseResult): void {
|
||||
for (const artifact of result.artifacts_created) {
|
||||
this.log(`Artifact created: ${artifact}`);
|
||||
}
|
||||
|
||||
if (result.decisions_made > 0 && this.decisionEngine) {
|
||||
this.decisionEngine.makeHighConfidenceDecision(
|
||||
`Agent reported ${result.decisions_made} decision(s) during ${result.stage}`,
|
||||
`Decisions recorded from ${result.stage} stage execution`,
|
||||
"general",
|
||||
[]
|
||||
);
|
||||
}
|
||||
|
||||
if (result.escalations_raised > 0 && this.escalationProtocol) {
|
||||
this.escalationProtocol.escalate({
|
||||
type: "low_confidence_decision",
|
||||
phase: String(this.pipelineState!.current_phase),
|
||||
description: `Agent reported ${result.escalations_raised} escalation(s) during ${result.stage}`,
|
||||
context: `Stage ${result.stage} raised escalations during execution`,
|
||||
options: [
|
||||
{ id: "proceed", label: "Proceed", description: "Continue pipeline execution", recommended: true },
|
||||
{ id: "halt", label: "Halt", description: "Stop pipeline and await manual review", recommended: false },
|
||||
],
|
||||
default_option_id: "proceed",
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
private deriveTotalPhases(): number {
|
||||
if (!this.ciFiles) return 1;
|
||||
const roadmap = this.ciFiles.readRoadmapMd();
|
||||
if (!roadmap || roadmap.phases.length === 0) return 1;
|
||||
return roadmap.phases.length;
|
||||
}
|
||||
|
||||
private performPhaseBoundaryCheckpoint(context: AgentContext): void {
|
||||
this.log(`Phase boundary checkpoint for phase ${this.pipelineState!.current_phase}`);
|
||||
|
||||
if (this.config.git.auto_commit && this.gitContext!.isGitRepo()) {
|
||||
try {
|
||||
const message = `chore(P${String(this.pipelineState!.current_phase).padStart(2, "0")}): phase boundary checkpoint\n\n---ci---\nphase: ${this.pipelineState!.current_phase}\nmilestone: ${this.currentMilestone}\nstatus: complete\n---/ci---`;
|
||||
execSync(`git add -A && git commit -m "${message.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
} catch (err) {
|
||||
this.warn(`Phase boundary commit failed: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
|
||||
if (this.ciFiles) {
|
||||
this.ciFiles.updatePhaseStatus(this.pipelineState!.current_phase, "complete");
|
||||
|
||||
const reqs = this.ciFiles.readRequirementsMd();
|
||||
if (reqs) {
|
||||
for (const t of reqs.traceability) {
|
||||
if (t.phase === this.pipelineState!.current_phase && t.status === "in_progress") {
|
||||
this.ciFiles.updateRequirementStatus(t.requirement, "complete");
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (this.gitContext) {
|
||||
const verifiedState = this.gitContext.reconstructState();
|
||||
this.log(`Verified state: phase=${verifiedState.currentPhase}, milestone=${verifiedState.currentMilestone}, stage=${verifiedState.currentStage}`);
|
||||
}
|
||||
}
|
||||
|
||||
private async executeStageWithRecovery(
|
||||
stage: PipelineStage,
|
||||
context: AgentContext
|
||||
): Promise<PhaseResult> {
|
||||
try {
|
||||
const result = await this.executeStage(stage, context);
|
||||
if (result.success) return result;
|
||||
} catch (err) {
|
||||
this.warn(`First attempt failed for ${stage}: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
|
||||
this.log(`Retrying stage ${stage}...`);
|
||||
try {
|
||||
const result = await this.executeStage(stage, context);
|
||||
if (result.success) return result;
|
||||
} catch (err) {
|
||||
this.warn(`Retry failed for ${stage}: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
|
||||
if (context.backend) {
|
||||
this.log(`Attempting plan revision for failed stage ${stage}...`);
|
||||
try {
|
||||
const planner = getAgent("planner");
|
||||
const gitContext = this.buildGitAgentContext(context);
|
||||
const planResult = await planner.execute({
|
||||
...gitContext,
|
||||
specification: `Plan revision needed: stage ${stage} failed twice. Original error context: phase ${this.pipelineState!.current_phase}`,
|
||||
});
|
||||
if (planResult.success) {
|
||||
this.log(`Plan revision succeeded, retrying ${stage} with revised plan...`);
|
||||
try {
|
||||
const result = await this.executeStage(stage, context);
|
||||
if (result.success) return result;
|
||||
} catch (err) {
|
||||
this.warn(`Post-revision retry failed for ${stage}: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
} catch (err) {
|
||||
this.warn(`Plan revision failed: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
|
||||
if (this.escalationProtocol) {
|
||||
this.escalationProtocol.escalate({
|
||||
type: "verification_failure",
|
||||
phase: String(this.pipelineState!.current_phase),
|
||||
description: `Stage ${stage} failed after retry and plan revision attempts`,
|
||||
context: `All recovery attempts exhausted for stage ${stage} in phase ${this.pipelineState!.current_phase}`,
|
||||
options: [
|
||||
{ id: "skip", label: "Skip stage", description: "Continue pipeline skipping this stage", recommended: true },
|
||||
{ id: "abort", label: "Abort pipeline", description: "Stop the entire pipeline", recommended: false },
|
||||
],
|
||||
default_option_id: "skip",
|
||||
});
|
||||
}
|
||||
|
||||
return {
|
||||
phase: this.pipelineState!.current_phase,
|
||||
stage,
|
||||
success: false,
|
||||
artifacts_created: [],
|
||||
decisions_made: 0,
|
||||
escalations_raised: 1,
|
||||
duration_ms: 0,
|
||||
error: `Stage ${stage} failed after recovery attempts`,
|
||||
};
|
||||
}
|
||||
|
||||
private async executeStage(
|
||||
stage: PipelineStage,
|
||||
context: AgentContext
|
||||
): Promise<PhaseResult> {
|
||||
const stageStart = Date.now();
|
||||
const agentName = OrchestratorAgent.STAGE_AGENT_MAP[stage];
|
||||
const agentNames = OrchestratorAgent.STAGE_AGENT_MAP[stage];
|
||||
|
||||
if (agentName && context.backend) {
|
||||
this.log(`Delegating ${stage} to ${agentName} agent via backend...`);
|
||||
if (agentNames && agentNames.length > 0 && context.backend) {
|
||||
this.log(`Delegating ${stage} to ${agentNames.join(", ")} agent(s) via backend...`);
|
||||
try {
|
||||
const agent = getAgent(agentName);
|
||||
const result = await agent.execute(context);
|
||||
let primaryResult: AgentResult | null = null;
|
||||
const allArtifacts: string[] = [];
|
||||
let totalDecisions = 0;
|
||||
let totalEscalations = 0;
|
||||
let lastError: string | undefined;
|
||||
|
||||
for (let i = 0; i < agentNames.length; i++) {
|
||||
const agentName = agentNames[i];
|
||||
const agent = getAgent(agentName);
|
||||
const gitContext = this.buildGitAgentContext(context);
|
||||
|
||||
if (i === 0) {
|
||||
const result = await agent.execute(gitContext);
|
||||
primaryResult = result;
|
||||
if (Array.isArray(result.artifacts_created)) {
|
||||
allArtifacts.push(...result.artifacts_created);
|
||||
}
|
||||
totalDecisions += result.decisions;
|
||||
totalEscalations += result.escalations;
|
||||
|
||||
if (!result.success) {
|
||||
this.warn(`Primary agent ${agentName} failed for ${stage}`);
|
||||
return {
|
||||
phase: this.pipelineState!.current_phase,
|
||||
stage,
|
||||
success: false,
|
||||
artifacts_created: allArtifacts,
|
||||
decisions_made: totalDecisions,
|
||||
escalations_raised: totalEscalations,
|
||||
duration_ms: Date.now() - stageStart,
|
||||
error: result.error || `Primary agent ${agentName} failed`,
|
||||
};
|
||||
}
|
||||
} else {
|
||||
try {
|
||||
const reviewContext: AgentContext = {
|
||||
...gitContext,
|
||||
specification: `${context.specification}\n\nPrimary agent (${agentNames[0]}) completed. Review context:\n- Success: ${primaryResult!.success}\n- Output: ${primaryResult!.output}\n- Artifacts: ${Array.isArray(primaryResult!.artifacts_created) ? primaryResult!.artifacts_created.join(", ") : String(primaryResult!.artifacts_created)}`,
|
||||
};
|
||||
const result = await agent.execute(reviewContext);
|
||||
if (Array.isArray(result.artifacts_created)) {
|
||||
allArtifacts.push(...result.artifacts_created);
|
||||
}
|
||||
totalDecisions += result.decisions;
|
||||
totalEscalations += result.escalations;
|
||||
|
||||
if (!result.success) {
|
||||
this.warn(`Review agent ${agentName} reported issues for ${stage}: ${result.error || "unspecified"}`);
|
||||
lastError = result.error;
|
||||
}
|
||||
} catch (err) {
|
||||
this.warn(`Review agent ${agentName} failed for ${stage}: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return {
|
||||
phase: this.pipelineState!.current_phase,
|
||||
stage,
|
||||
success: result.success,
|
||||
artifacts_created: Array.isArray(result.artifacts_created) ? result.artifacts_created : [],
|
||||
decisions_made: result.decisions,
|
||||
escalations_raised: result.escalations,
|
||||
success: primaryResult?.success ?? false,
|
||||
artifacts_created: allArtifacts,
|
||||
decisions_made: totalDecisions,
|
||||
escalations_raised: totalEscalations,
|
||||
duration_ms: Date.now() - stageStart,
|
||||
error: result.error,
|
||||
error: lastError,
|
||||
};
|
||||
} catch (err) {
|
||||
if (err instanceof BackendUnavailableError) {
|
||||
this.warn(`Backend unavailable for ${stage}, falling back to mechanical execution`);
|
||||
} else {
|
||||
this.warn(`Agent ${agentName} failed for ${stage}: ${err instanceof Error ? err.message : String(err)}`);
|
||||
this.warn(`Agents failed for ${stage}: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -207,11 +434,10 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
});
|
||||
|
||||
this.log("Init commit prepared with specification in ---ci--- block");
|
||||
artifactsCreated.push(".ci/config.json");
|
||||
artifactsCreated.push(".ciagent/config.json");
|
||||
|
||||
if (this.config.git.auto_commit && this.gitContext!.isGitRepo()) {
|
||||
try {
|
||||
const { execSync } = await import("node:child_process");
|
||||
this.ciFiles!.writeProjectMd({
|
||||
name: spec.objective.slice(0, 30),
|
||||
coreValue: spec.objective,
|
||||
@@ -296,10 +522,9 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
1,
|
||||
this.currentMilestone,
|
||||
"initial domain research",
|
||||
["Research completed. Key findings in .ci/ARCHITECTURE.md and .ci/PROJECT.md updates."]
|
||||
["Research completed. Key findings in .ciagent/ARCHITECTURE.md and .ciagent/PROJECT.md updates."]
|
||||
);
|
||||
try {
|
||||
const { execSync } = await import("node:child_process");
|
||||
execSync(`git add -A && git commit -m "${researchCommit.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
@@ -310,7 +535,7 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
}
|
||||
|
||||
this.pipelineState!.research_completed = true;
|
||||
artifactsCreated.push(".ci/ARCHITECTURE.md");
|
||||
artifactsCreated.push(".ciagent/ARCHITECTURE.md");
|
||||
break;
|
||||
}
|
||||
|
||||
@@ -342,6 +567,38 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
this.pipelineState!.execute_completed = true;
|
||||
break;
|
||||
|
||||
case "test": {
|
||||
this.log("Running tests...");
|
||||
if (!context.backend) {
|
||||
this.log("No backend available — running mechanical test fallback via npm test");
|
||||
try {
|
||||
const testOutput = execSync("npm test", {
|
||||
cwd: context.project_path,
|
||||
encoding: "utf-8",
|
||||
stdio: ["pipe", "pipe", "pipe"],
|
||||
timeout: 120000,
|
||||
});
|
||||
this.log("npm test passed");
|
||||
this.pipelineState!.test_completed = true;
|
||||
artifactsCreated.push("test-results");
|
||||
} catch (err) {
|
||||
const errMsg = err instanceof Error ? err.message : String(err);
|
||||
this.warn(`npm test failed: ${errMsg}`);
|
||||
return {
|
||||
phase: this.pipelineState!.current_phase,
|
||||
stage: "test",
|
||||
success: false,
|
||||
artifacts_created: artifactsCreated,
|
||||
decisions_made: decisionsMade,
|
||||
escalations_raised: escalationsRaised,
|
||||
duration_ms: Date.now() - stageStart,
|
||||
error: `Test stage failed: ${errMsg}`,
|
||||
};
|
||||
}
|
||||
}
|
||||
break;
|
||||
}
|
||||
|
||||
case "verify": {
|
||||
this.log("Running verification...");
|
||||
|
||||
@@ -372,7 +629,6 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
requirements: { covered: [], partial: [] },
|
||||
});
|
||||
try {
|
||||
const { execSync } = await import("node:child_process");
|
||||
execSync(`git add -A && git commit -m "${verifyCommit.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
@@ -398,7 +654,6 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
taskNames: [],
|
||||
});
|
||||
try {
|
||||
const { execSync } = await import("node:child_process");
|
||||
execSync(`git add -A && git commit -m "${completionCommit.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
@@ -408,6 +663,30 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
}
|
||||
}
|
||||
|
||||
const versionTag = `${this.currentMilestone}-P${String(this.pipelineState!.current_phase).padStart(2, "0")}`;
|
||||
try {
|
||||
execSync(`git tag "${versionTag}"`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
this.log(`Created version tag: ${versionTag}`);
|
||||
artifactsCreated.push(`tag:${versionTag}`);
|
||||
} catch (err) {
|
||||
this.warn(`Version tag creation failed: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
|
||||
if (this.config.git.auto_push && this.gitContext!.isGitRepo()) {
|
||||
try {
|
||||
execSync(`git push origin ${versionTag}`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
this.log(`Pushed version tag: ${versionTag}`);
|
||||
} catch (err) {
|
||||
this.warn(`Version tag push failed: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
|
||||
break;
|
||||
}
|
||||
}
|
||||
@@ -425,7 +704,7 @@ export class OrchestratorAgent extends BaseAgent {
|
||||
|
||||
private generateCompletionReport(): string {
|
||||
const lines: string[] = [
|
||||
"# CI Completion Report",
|
||||
"# CIAgent Completion Report",
|
||||
"",
|
||||
`✓ Pipeline completed successfully (git-native)`,
|
||||
"",
|
||||
|
||||
@@ -0,0 +1,167 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { PlannerAgent } from "../agents/planner.js";
|
||||
import { AgentContext } from "../agents/base.js";
|
||||
import { IntelligenceBackend, BackendRequest, BackendResult } from "../backends/types.js";
|
||||
import { Decision } from "../types/decisions.js";
|
||||
import { Escalation } from "../types/escalation.js";
|
||||
import { emptyTokenUsage } from "../backends/types.js";
|
||||
|
||||
class MockBackend implements IntelligenceBackend {
|
||||
readonly name = "mock";
|
||||
readonly type = "llm" as const;
|
||||
async isAvailable(): Promise<boolean> { return true; }
|
||||
async execute(request: BackendRequest): Promise<BackendResult> {
|
||||
return {
|
||||
success: true,
|
||||
output: `Mock backend executed: ${request.task.slice(0, 50)}`,
|
||||
artifacts: [],
|
||||
decisions: [],
|
||||
escalations: [],
|
||||
usage: emptyTokenUsage(),
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-planner-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
function makeContext(dir: string, backend?: IntelligenceBackend): AgentContext {
|
||||
return {
|
||||
project_path: dir,
|
||||
phase: 1,
|
||||
stage: "plan",
|
||||
specification: "Build a REST API for task management",
|
||||
config_path: path.join(dir, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
}
|
||||
|
||||
function setupCIAgentDir(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), "{}");
|
||||
}
|
||||
|
||||
function writeRequirementsMd(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
const content = [
|
||||
"# Requirements",
|
||||
"",
|
||||
"## v1 Requirements",
|
||||
"",
|
||||
"### Core",
|
||||
"",
|
||||
"- [ ] **REQ-01**: User authentication",
|
||||
"- [ ] **REQ-02**: Task CRUD operations",
|
||||
"- [ ] **REQ-03**: Real-time notifications",
|
||||
"",
|
||||
"## Traceability",
|
||||
"",
|
||||
"| Requirement | Phase | Status |",
|
||||
"|-------------|-------|--------|",
|
||||
"| REQ-01 | Phase 1 | in_progress |",
|
||||
"| REQ-02 | Phase 1 | pending |",
|
||||
"| REQ-03 | Phase 1 | blocked |",
|
||||
].join("\n");
|
||||
fs.writeFileSync(path.join(ciDir, "REQUIREMENTS.md"), content);
|
||||
}
|
||||
|
||||
function writeRoadmapMd(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
const content = [
|
||||
"# Roadmap",
|
||||
"",
|
||||
"## Overview",
|
||||
"",
|
||||
"Task management API roadmap",
|
||||
"",
|
||||
"## Phases",
|
||||
"",
|
||||
"- [ ] **Phase 1: Authentication** - Implement auth",
|
||||
"",
|
||||
"## Phase Details",
|
||||
"",
|
||||
"### Phase 1: Authentication",
|
||||
"**Goal**: Implement user authentication",
|
||||
"**Depends on**: Nothing",
|
||||
"**Requirements**: REQ-01, REQ-02",
|
||||
"**Success Criteria**:",
|
||||
"1. .ciagent/REQUIREMENTS.md exists",
|
||||
"**Status**: in_progress",
|
||||
].join("\n");
|
||||
fs.writeFileSync(path.join(ciDir, "ROADMAP.md"), content);
|
||||
}
|
||||
|
||||
describe("PlannerAgent", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = createTempDir();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
it("returns honest failure without backend when no requirements or roadmap", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
const planner = new PlannerAgent();
|
||||
const result = await planner.execute(makeContext(dir));
|
||||
expect(result.success).toBe(false);
|
||||
expect(result.error).toContain("No requirements or roadmap");
|
||||
});
|
||||
|
||||
it("creates PLAN.md from REQUIREMENTS.md without backend", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
writeRequirementsMd(dir);
|
||||
writeRoadmapMd(dir);
|
||||
|
||||
const planner = new PlannerAgent();
|
||||
const result = await planner.execute(makeContext(dir));
|
||||
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("plan");
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "PLAN.md"))).toBe(true);
|
||||
});
|
||||
|
||||
it("PLAN.md contains phase goal and tasks", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
writeRequirementsMd(dir);
|
||||
writeRoadmapMd(dir);
|
||||
|
||||
const planner = new PlannerAgent();
|
||||
await planner.execute(makeContext(dir));
|
||||
|
||||
const planContent = fs.readFileSync(path.join(dir, ".ciagent", "PLAN.md"), "utf-8");
|
||||
expect(planContent).toContain("Phase 1 Plan");
|
||||
expect(planContent).toContain("Phase Goal");
|
||||
expect(planContent).toContain("Tasks");
|
||||
});
|
||||
|
||||
it("delegates to backend when available", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
const mockBackend = new MockBackend();
|
||||
const planner = new PlannerAgent();
|
||||
const result = await planner.execute(makeContext(dir, mockBackend));
|
||||
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("Mock backend executed");
|
||||
});
|
||||
|
||||
it("has correct agent name", () => {
|
||||
const planner = new PlannerAgent();
|
||||
expect(planner.name).toBe("planner");
|
||||
});
|
||||
|
||||
it("has correct workflow", () => {
|
||||
const planner = new PlannerAgent();
|
||||
expect(planner.workflow).toBe("plan");
|
||||
});
|
||||
});
|
||||
+323
-9
@@ -1,4 +1,27 @@
|
||||
import { BaseAgent, AgentContext, AgentResult } from "./base.js";
|
||||
import { CIAgentFiles, RequirementsMd, RoadmapMd, ArchitectureMd } from "../core/ciagent-files.js";
|
||||
import { GitContext } from "../core/git-context.js";
|
||||
import { CommitBuilder } from "../core/commit-builder.js";
|
||||
import { writeFile, readFile, ensureDir } from "../utils/file.js";
|
||||
import { execSync } from "node:child_process";
|
||||
import * as path from "node:path";
|
||||
|
||||
export interface PlannerResult {
|
||||
success: boolean;
|
||||
planCount: number;
|
||||
waves: { wave: number; plans: string[] }[];
|
||||
decisions: number;
|
||||
error?: string;
|
||||
}
|
||||
|
||||
interface PlanEntry {
|
||||
name: string;
|
||||
wave: number;
|
||||
requirements: string[];
|
||||
dependsOn: string[];
|
||||
tasks: string[];
|
||||
mustHaves: string[];
|
||||
}
|
||||
|
||||
export class PlannerAgent extends BaseAgent {
|
||||
readonly name = "planner";
|
||||
@@ -8,21 +31,312 @@ export class PlannerAgent extends BaseAgent {
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
const start = Date.now();
|
||||
this.log("Creating phase plan...");
|
||||
|
||||
if (context.backend) {
|
||||
const result = await this.executeViaBackend(
|
||||
context,
|
||||
`Create a phase plan for stage ${context.stage}, phase ${context.phase}. Specification: ${context.specification}`
|
||||
);
|
||||
const taskPrompt = await this.buildBackendTaskPrompt(context);
|
||||
const result = await this.executeViaBackend(context, taskPrompt);
|
||||
return { ...result, duration_ms: Date.now() - start };
|
||||
}
|
||||
|
||||
return this.executeMechanical(context, start);
|
||||
}
|
||||
|
||||
private async buildBackendTaskPrompt(context: AgentContext): Promise<string> {
|
||||
const ciFiles = new CIAgentFiles(context.project_path);
|
||||
const parts: string[] = [
|
||||
`Create a phase plan for stage ${context.stage}, phase ${context.phase}.`,
|
||||
"",
|
||||
"## Project Context",
|
||||
];
|
||||
|
||||
const roadmap = ciFiles.readRoadmapMd();
|
||||
if (roadmap) {
|
||||
const currentPhase = roadmap.phases.find((p) => p.number === context.phase);
|
||||
if (currentPhase) {
|
||||
parts.push("", "### Phase Goal", currentPhase.description);
|
||||
parts.push("", "### Phase Requirements", currentPhase.requirements.join(", ") || "None specified");
|
||||
parts.push("", "### Phase Dependencies", currentPhase.dependsOn.length > 0 ? currentPhase.dependsOn.map((d) => `Phase ${d}`).join(", ") : "None");
|
||||
parts.push("", "### Success Criteria", ...currentPhase.successCriteria.map((sc) => `- ${sc}`));
|
||||
}
|
||||
}
|
||||
|
||||
const requirements = ciFiles.readRequirementsMd();
|
||||
if (requirements) {
|
||||
const phaseReqs = requirements.traceability.filter((t) => t.phase === context.phase);
|
||||
if (phaseReqs.length > 0) {
|
||||
parts.push("", "### Requirements for Phase", ...phaseReqs.map((t) => `- ${t.requirement} (${t.status})`));
|
||||
}
|
||||
}
|
||||
|
||||
const architecture = ciFiles.readArchitectureMd();
|
||||
if (architecture) {
|
||||
parts.push("", "### Architecture Boundaries", ...architecture.components.map((c) => `- ${c.name}: ${c.boundaries}`));
|
||||
parts.push("", "### Build Order", ...architecture.buildOrder.map((bo) => `${bo}`));
|
||||
}
|
||||
|
||||
parts.push("", "## Specification", context.specification || "No specification provided");
|
||||
|
||||
return parts.join("\n");
|
||||
}
|
||||
|
||||
private executeMechanical(context: AgentContext, start: number): AgentResult {
|
||||
const ciFiles = new CIAgentFiles(context.project_path);
|
||||
ciFiles.ensureCIDir();
|
||||
|
||||
const requirements = ciFiles.readRequirementsMd();
|
||||
const roadmap = ciFiles.readRoadmapMd();
|
||||
const architecture = ciFiles.readArchitectureMd();
|
||||
|
||||
if (!requirements && !roadmap) {
|
||||
return {
|
||||
success: false,
|
||||
output: "Planning requires either .ciagent/REQUIREMENTS.md or .ciagent/ROADMAP.md. Initialize the project first.",
|
||||
artifacts_created: [],
|
||||
decisions: 0,
|
||||
escalations: 0,
|
||||
duration_ms: Date.now() - start,
|
||||
error: "No requirements or roadmap found for mechanical planning",
|
||||
};
|
||||
}
|
||||
|
||||
let gitLogSummary = "";
|
||||
try {
|
||||
gitLogSummary = execSync("git log --max-count=20 --oneline", {
|
||||
cwd: context.project_path,
|
||||
encoding: "utf-8",
|
||||
stdio: ["pipe", "pipe", "pipe"],
|
||||
}).trim();
|
||||
} catch {
|
||||
gitLogSummary = "(no git history available)";
|
||||
}
|
||||
|
||||
const phaseGoal = this.extractPhaseGoal(roadmap, context.phase);
|
||||
const phaseRequirements = this.extractPhaseRequirements(requirements, context.phase);
|
||||
const componentBoundaries = architecture ? architecture.components.map((c) => c.name) : [];
|
||||
|
||||
const plans = this.buildPlans(phaseRequirements, componentBoundaries, context.phase);
|
||||
|
||||
const planFileContent = this.formatPlanFile(context.phase, phaseGoal, plans);
|
||||
|
||||
const planFilePath = path.join(context.project_path, ".ciagent", "PLAN.md");
|
||||
ensureDir(path.dirname(planFilePath));
|
||||
writeFile(planFilePath, planFileContent);
|
||||
|
||||
const decisionCount = plans.length > 0 ? 1 : 0;
|
||||
|
||||
if (this.shouldCommit(context)) {
|
||||
try {
|
||||
const commitMessage = CommitBuilder.buildTaskCommit({
|
||||
type: "docs",
|
||||
phase: context.phase,
|
||||
milestone: "v1.0",
|
||||
plan: "01",
|
||||
task: "01-01",
|
||||
subject: `create ${plans.length} phase plans`,
|
||||
status: "plan",
|
||||
decisions: decisionCount > 0 ? [{
|
||||
id: "D-001",
|
||||
decision: `Decomposed phase ${context.phase} into ${plans.length} vertical-slice plans`,
|
||||
rationale: "Requirements grouped by dependency analysis — independent requirements in wave 1, dependent in wave 2+",
|
||||
confidence: 0.75,
|
||||
alternatives: ["single monolithic plan", "per-requirement plans"],
|
||||
}] : undefined,
|
||||
});
|
||||
execSync(`git add -A && git commit -m "${commitMessage.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
} catch {
|
||||
this.warn("Plan commit failed");
|
||||
}
|
||||
}
|
||||
|
||||
const waves = this.groupPlansByWave(plans);
|
||||
const plannerResult: PlannerResult = {
|
||||
success: true,
|
||||
planCount: plans.length,
|
||||
waves,
|
||||
decisions: decisionCount,
|
||||
};
|
||||
|
||||
return {
|
||||
success: false,
|
||||
output: "Planning requires an intelligence backend. Configure one with: ci init --backend",
|
||||
artifacts_created: [],
|
||||
decisions: 0,
|
||||
success: true,
|
||||
output: `Created ${plans.length} plan(s) across ${waves.length} wave(s) for phase ${context.phase}`,
|
||||
artifacts_created: [".ciagent/PLAN.md"],
|
||||
decisions: decisionCount,
|
||||
escalations: 0,
|
||||
duration_ms: Date.now() - start,
|
||||
error: "No intelligence backend available",
|
||||
};
|
||||
}
|
||||
|
||||
private extractPhaseGoal(roadmap: RoadmapMd | null, phase: number): string {
|
||||
if (!roadmap) return "No roadmap available";
|
||||
const phaseEntry = roadmap.phases.find((p) => p.number === phase);
|
||||
if (phaseEntry) return `${phaseEntry.name}: ${phaseEntry.description}`;
|
||||
return `Phase ${phase} (no roadmap entry)`;
|
||||
}
|
||||
|
||||
private extractPhaseRequirements(requirements: RequirementsMd | null, phase: number): Array<{ id: string; description: string; phase: number; status: string }> {
|
||||
if (!requirements) return [];
|
||||
return requirements.traceability
|
||||
.filter((t) => t.phase === phase)
|
||||
.map((t) => {
|
||||
let description = t.requirement;
|
||||
for (const cat of [...requirements.v1, ...requirements.v2]) {
|
||||
const item = cat.items.find((i) => i.id === t.requirement);
|
||||
if (item) {
|
||||
description = `${t.requirement}: ${item.description}`;
|
||||
break;
|
||||
}
|
||||
}
|
||||
return { id: t.requirement, description, phase: t.phase, status: t.status };
|
||||
});
|
||||
}
|
||||
|
||||
private buildPlans(
|
||||
phaseRequirements: Array<{ id: string; description: string; phase: number; status: string }>,
|
||||
componentBoundaries: string[],
|
||||
phase: number
|
||||
): PlanEntry[] {
|
||||
if (phaseRequirements.length === 0) {
|
||||
return [{
|
||||
name: `Phase ${phase} Core Implementation`,
|
||||
wave: 1,
|
||||
requirements: [],
|
||||
dependsOn: [],
|
||||
tasks: [`Implement phase ${phase} deliverables as specified in ROADMAP.md`],
|
||||
mustHaves: [`Phase ${phase} deliverables exist and pass verification`],
|
||||
}];
|
||||
}
|
||||
|
||||
const independentReqs = phaseRequirements.filter((r) => r.status !== "blocked");
|
||||
const blockedReqs = phaseRequirements.filter((r) => r.status === "blocked");
|
||||
|
||||
const plans: PlanEntry[] = [];
|
||||
|
||||
if (independentReqs.length > 0) {
|
||||
const taskChunks = this.chunkByComponent(independentReqs, componentBoundaries);
|
||||
for (const chunk of taskChunks) {
|
||||
plans.push({
|
||||
name: this.inferPlanName(chunk, phase),
|
||||
wave: 1,
|
||||
requirements: chunk.map((r) => r.id),
|
||||
dependsOn: [],
|
||||
tasks: chunk.map((r) => {
|
||||
const desc = r.description.split(": ").slice(1).join(": ") || r.description;
|
||||
return desc !== r.id ? `Implement ${r.id}: ${desc}` : `Implement ${r.id}`;
|
||||
}),
|
||||
mustHaves: chunk.map((r) => `${r.id} implemented and testable`),
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
if (blockedReqs.length > 0) {
|
||||
const taskChunks = this.chunkByComponent(blockedReqs, componentBoundaries);
|
||||
for (const chunk of taskChunks) {
|
||||
plans.push({
|
||||
name: this.inferPlanName(chunk, phase),
|
||||
wave: plans.length > 0 ? Math.max(...plans.map((p) => p.wave)) + 1 : 2,
|
||||
requirements: chunk.map((r) => r.id),
|
||||
dependsOn: plans.slice(0, plans.length > 0 ? 1 : 0).map((p) => p.name),
|
||||
tasks: chunk.map((r) => {
|
||||
const desc = r.description.split(": ").slice(1).join(": ") || r.description;
|
||||
return desc !== r.id ? `Implement ${r.id}: ${desc}` : `Implement ${r.id}`;
|
||||
}),
|
||||
mustHaves: chunk.map((r) => `${r.id} implemented and testable`),
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
if (plans.length === 0) {
|
||||
plans.push({
|
||||
name: `Phase ${phase} Default`,
|
||||
wave: 1,
|
||||
requirements: [],
|
||||
dependsOn: [],
|
||||
tasks: [`Implement phase ${phase} deliverables`],
|
||||
mustHaves: [`Phase ${phase} deliverables pass verification`],
|
||||
});
|
||||
}
|
||||
|
||||
return plans;
|
||||
}
|
||||
|
||||
private chunkByComponent(
|
||||
reqs: Array<{ id: string; description: string; phase: number; status: string }>,
|
||||
_componentBoundaries: string[]
|
||||
): Array<Array<{ id: string; description: string; phase: number; status: string }>> {
|
||||
if (reqs.length <= 3) return [reqs];
|
||||
const chunks: Array<Array<{ id: string; description: string; phase: number; status: string }>> = [];
|
||||
const chunkSize = Math.ceil(reqs.length / Math.ceil(reqs.length / 3));
|
||||
for (let i = 0; i < reqs.length; i += chunkSize) {
|
||||
chunks.push(reqs.slice(i, i + chunkSize));
|
||||
}
|
||||
return chunks;
|
||||
}
|
||||
|
||||
private inferPlanName(chunk: Array<{ id: string; description: string; phase: number; status: string }>, phase: number): string {
|
||||
if (chunk.length === 1) return `Phase ${phase}: ${chunk[0].id}`;
|
||||
return `Phase ${phase}: ${chunk[0].id}–${chunk[chunk.length - 1].id}`;
|
||||
}
|
||||
|
||||
private groupPlansByWave(plans: PlanEntry[]): { wave: number; plans: string[] }[] {
|
||||
const waveMap = new Map<number, string[]>();
|
||||
for (const plan of plans) {
|
||||
const existing = waveMap.get(plan.wave) || [];
|
||||
existing.push(plan.name);
|
||||
waveMap.set(plan.wave, existing);
|
||||
}
|
||||
return Array.from(waveMap.entries())
|
||||
.sort((a, b) => a[0] - b[0])
|
||||
.map(([wave, names]) => ({ wave, plans: names }));
|
||||
}
|
||||
|
||||
private formatPlanFile(phase: number, phaseGoal: string, plans: PlanEntry[]): string {
|
||||
const lines: string[] = [
|
||||
`# Phase ${phase} Plan`,
|
||||
"",
|
||||
"## Phase Goal",
|
||||
phaseGoal,
|
||||
"",
|
||||
"## Plans",
|
||||
"",
|
||||
];
|
||||
|
||||
for (let i = 0; i < plans.length; i++) {
|
||||
const plan = plans[i];
|
||||
const planNum = i + 1;
|
||||
lines.push(`### Plan ${planNum}: ${plan.name}`);
|
||||
lines.push(`- Wave: ${plan.wave}`);
|
||||
if (plan.requirements.length > 0) {
|
||||
lines.push(`- Requirements: [${plan.requirements.join(", ")}]`);
|
||||
}
|
||||
if (plan.dependsOn.length > 0) {
|
||||
lines.push(`- Depends on: ${plan.dependsOn.join(", ")}`);
|
||||
}
|
||||
lines.push("- Tasks:");
|
||||
for (const task of plan.tasks) {
|
||||
lines.push(` 1. ${task}`);
|
||||
}
|
||||
lines.push("- Must-haves:");
|
||||
for (const mh of plan.mustHaves) {
|
||||
lines.push(` - [x] ${mh}`);
|
||||
}
|
||||
lines.push("");
|
||||
}
|
||||
|
||||
return lines.join("\n");
|
||||
}
|
||||
|
||||
private shouldCommit(context: AgentContext): boolean {
|
||||
try {
|
||||
execSync("git rev-parse --is-inside-work-tree", {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
return true;
|
||||
} catch {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,208 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { ResearcherAgent } from "../agents/researcher.js";
|
||||
import { AgentContext } from "../agents/base.js";
|
||||
import { IntelligenceBackend, BackendRequest, BackendResult } from "../backends/types.js";
|
||||
import { emptyTokenUsage } from "../backends/types.js";
|
||||
|
||||
class MockBackend implements IntelligenceBackend {
|
||||
readonly name = "mock";
|
||||
readonly type = "llm" as const;
|
||||
async isAvailable(): Promise<boolean> { return true; }
|
||||
async execute(request: BackendRequest): Promise<BackendResult> {
|
||||
return {
|
||||
success: true,
|
||||
output: `Mock backend executed: ${request.task.slice(0, 50)}`,
|
||||
artifacts: [],
|
||||
decisions: [],
|
||||
escalations: [],
|
||||
usage: emptyTokenUsage(),
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-researcher-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
function makeContext(dir: string, backend?: IntelligenceBackend): AgentContext {
|
||||
return {
|
||||
project_path: dir,
|
||||
phase: 1,
|
||||
stage: "research",
|
||||
specification: "Build a REST API for task management",
|
||||
config_path: path.join(dir, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
}
|
||||
|
||||
function setupCIAgentDir(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), '{"projects":[],"active_project":""}');
|
||||
}
|
||||
|
||||
function writeProjectMd(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
const content = [
|
||||
"# Task API",
|
||||
"",
|
||||
"## What This Is",
|
||||
"",
|
||||
"A REST API for managing tasks",
|
||||
"",
|
||||
"## Requirements",
|
||||
"",
|
||||
"### Validated",
|
||||
"",
|
||||
"- ✓ User authentication",
|
||||
"",
|
||||
"### Active",
|
||||
"",
|
||||
"- [ ] Task CRUD",
|
||||
"",
|
||||
"### Out of Scope",
|
||||
"",
|
||||
"- Admin dashboard",
|
||||
"",
|
||||
"## Context",
|
||||
"",
|
||||
"Node.js project",
|
||||
"",
|
||||
"## Constraints",
|
||||
"",
|
||||
"- Must use Node.js",
|
||||
"",
|
||||
"## Key Decisions",
|
||||
"",
|
||||
"| Decision | Rationale | Outcome |",
|
||||
"|----------|-----------|---------|",
|
||||
].join("\n");
|
||||
fs.writeFileSync(path.join(ciDir, "PROJECT.md"), content);
|
||||
}
|
||||
|
||||
function writeArchitectureMd(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
const content = [
|
||||
"# Architecture",
|
||||
"",
|
||||
"## Overview",
|
||||
"",
|
||||
"Task management system architecture",
|
||||
"",
|
||||
"## Components",
|
||||
"",
|
||||
"### Core",
|
||||
"- **Description**: Core module",
|
||||
"- **Boundaries**: src/core/ — internal module",
|
||||
"- **Depends on**: None",
|
||||
"",
|
||||
"## Data Flow",
|
||||
"",
|
||||
"Request → Handler → Service → Database",
|
||||
"",
|
||||
"## Build Order",
|
||||
"",
|
||||
"1. Build core module",
|
||||
].join("\n");
|
||||
fs.writeFileSync(path.join(ciDir, "ARCHITECTURE.md"), content);
|
||||
}
|
||||
|
||||
function setupSourceDir(dir: string): void {
|
||||
const srcDir = path.join(dir, "src");
|
||||
fs.mkdirSync(srcDir, { recursive: true });
|
||||
fs.mkdirSync(path.join(srcDir, "core"), { recursive: true });
|
||||
fs.mkdirSync(path.join(srcDir, "agents"), { recursive: true });
|
||||
fs.writeFileSync(path.join(srcDir, "core", "index.ts"), "export {};\n");
|
||||
fs.writeFileSync(path.join(srcDir, "agents", "base.ts"), "export {};\n");
|
||||
}
|
||||
|
||||
describe("ResearcherAgent", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = createTempDir();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
it("reads .ciagent/ files without backend", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
writeProjectMd(dir);
|
||||
writeArchitectureMd(dir);
|
||||
|
||||
const researcher = new ResearcherAgent();
|
||||
const result = await researcher.execute(makeContext(dir));
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("findingsCount");
|
||||
});
|
||||
|
||||
it("only modifies .ciagent/ files", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
writeProjectMd(dir);
|
||||
writeArchitectureMd(dir);
|
||||
setupSourceDir(dir);
|
||||
|
||||
const srcDir = path.join(dir, "src");
|
||||
const filesBefore = new Set<string>();
|
||||
function collectFiles(d: string): void {
|
||||
for (const entry of fs.readdirSync(d, { withFileTypes: true })) {
|
||||
const full = path.join(d, entry.name);
|
||||
if (entry.isDirectory() && entry.name !== "node_modules") {
|
||||
collectFiles(full);
|
||||
} else {
|
||||
filesBefore.add(full);
|
||||
}
|
||||
}
|
||||
}
|
||||
collectFiles(srcDir);
|
||||
|
||||
const researcher = new ResearcherAgent();
|
||||
await researcher.execute(makeContext(dir));
|
||||
|
||||
collectFiles(srcDir);
|
||||
for (const f of filesBefore) {
|
||||
expect(fs.existsSync(f)).toBe(true);
|
||||
}
|
||||
});
|
||||
|
||||
it("updates ARCHITECTURE.md from source scan", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
writeProjectMd(dir);
|
||||
setupSourceDir(dir);
|
||||
|
||||
const researcher = new ResearcherAgent();
|
||||
const result = await researcher.execute(makeContext(dir));
|
||||
|
||||
if (result.success) {
|
||||
const parsed = JSON.parse(result.output);
|
||||
expect(parsed.filesUpdated).toContain(".ciagent/ARCHITECTURE.md");
|
||||
}
|
||||
});
|
||||
|
||||
it("delegates to backend when available", async () => {
|
||||
setupCIAgentDir(dir);
|
||||
const mockBackend = new MockBackend();
|
||||
const researcher = new ResearcherAgent();
|
||||
const result = await researcher.execute(makeContext(dir, mockBackend));
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("Mock backend executed");
|
||||
});
|
||||
|
||||
it("has correct agent name", () => {
|
||||
const researcher = new ResearcherAgent();
|
||||
expect(researcher.name).toBe("researcher");
|
||||
});
|
||||
|
||||
it("has correct workflow", () => {
|
||||
const researcher = new ResearcherAgent();
|
||||
expect(researcher.workflow).toBe("research");
|
||||
});
|
||||
});
|
||||
+240
-6
@@ -1,4 +1,20 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { BaseAgent, AgentContext, AgentResult } from "./base.js";
|
||||
import { GitContext } from "../core/git-context.js";
|
||||
import { CIAgentFiles, ArchitectureMd, ProjectMd } from "../core/ciagent-files.js";
|
||||
import { CommitBuilder } from "../core/commit-builder.js";
|
||||
import { CommitDecision } from "../types/commit-meta.js";
|
||||
import { fileExists, readFile } from "../utils/file.js";
|
||||
import { execSync } from "node:child_process";
|
||||
|
||||
export interface ResearcherResult {
|
||||
success: boolean;
|
||||
findingsCount: number;
|
||||
decisionsLogged: number;
|
||||
filesUpdated: string[];
|
||||
error?: string;
|
||||
}
|
||||
|
||||
export class ResearcherAgent extends BaseAgent {
|
||||
readonly name = "researcher";
|
||||
@@ -8,21 +24,239 @@ export class ResearcherAgent extends BaseAgent {
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
const start = Date.now();
|
||||
this.log("Researching domain...");
|
||||
|
||||
if (context.backend) {
|
||||
const result = await this.executeViaBackend(
|
||||
context,
|
||||
`Research the domain for: ${context.specification}`
|
||||
`Research the domain for phase ${context.phase}. Specification: ${context.specification}. Read git history (last 50 commits), .ciagent/PROJECT.md, .ciagent/ARCHITECTURE.md, .ciagent/REQUIREMENTS.md. Scan src/ directory structure. Generate findings about module boundaries, risks, and approach. Update .ciagent/ARCHITECTURE.md with component boundary conclusions. Update .ciagent/PROJECT.md key decisions if warranted. Commit findings with CommitBuilder.buildResearchCommit().`
|
||||
);
|
||||
return { ...result, duration_ms: Date.now() - start };
|
||||
}
|
||||
|
||||
const result = await this.runMechanicalResearch(context);
|
||||
const output = JSON.stringify(result, null, 2);
|
||||
|
||||
return {
|
||||
success: false,
|
||||
output: "Research requires an intelligence backend. Configure one with: ci init --backend",
|
||||
artifacts_created: [],
|
||||
decisions: 0,
|
||||
success: result.success,
|
||||
output,
|
||||
artifacts_created: result.filesUpdated,
|
||||
decisions: result.decisionsLogged,
|
||||
escalations: 0,
|
||||
duration_ms: Date.now() - start,
|
||||
error: "No intelligence backend available",
|
||||
error: result.error,
|
||||
};
|
||||
}
|
||||
|
||||
private async runMechanicalResearch(context: AgentContext): Promise<ResearcherResult> {
|
||||
try {
|
||||
const gitContext = new GitContext(context.project_path);
|
||||
const ciFiles = new CIAgentFiles(context.project_path);
|
||||
|
||||
const findings: string[] = [];
|
||||
const decisions: CommitDecision[] = [];
|
||||
const filesUpdated: string[] = [];
|
||||
|
||||
const commits = gitContext.getRecentCommits(50);
|
||||
if (commits.length > 0) {
|
||||
findings.push(`Analyzed ${commits.length} recent commits for project history`);
|
||||
const researchCommits = commits.filter(c => c.ci?.status === "research");
|
||||
if (researchCommits.length > 0) {
|
||||
findings.push(`Found ${researchCommits.length} prior research commits`);
|
||||
}
|
||||
}
|
||||
|
||||
const projectMd = ciFiles.readProjectMd();
|
||||
if (projectMd) {
|
||||
findings.push(`Project: ${projectMd.name} — core value: ${projectMd.coreValue.slice(0, 80)}`);
|
||||
findings.push(`Active requirements: ${projectMd.requirements.active.length}, validated: ${projectMd.requirements.validated.length}`);
|
||||
} else {
|
||||
findings.push("No PROJECT.md found — project context unavailable");
|
||||
}
|
||||
|
||||
const archMd = ciFiles.readArchitectureMd();
|
||||
if (archMd) {
|
||||
findings.push(`Architecture: ${archMd.components.length} components, ${archMd.buildOrder.length} build steps`);
|
||||
for (const comp of archMd.components) {
|
||||
findings.push(` Component: ${comp.name} — boundaries: ${comp.boundaries.slice(0, 60)}, deps: ${comp.dependsOn.join(", ") || "none"}`);
|
||||
}
|
||||
} else {
|
||||
findings.push("No ARCHITECTURE.md found — architecture analysis unavailable");
|
||||
}
|
||||
|
||||
const reqsMd = ciFiles.readRequirementsMd();
|
||||
if (reqsMd) {
|
||||
const totalReqs = reqsMd.traceability.length;
|
||||
const covered = reqsMd.traceability.filter(t => t.status === "complete").length;
|
||||
const phaseReqs = reqsMd.traceability.filter(t => t.phase === context.phase);
|
||||
findings.push(`Requirements: ${totalReqs} total, ${covered} complete, ${phaseReqs.length} for phase ${context.phase}`);
|
||||
}
|
||||
|
||||
const srcDir = path.join(context.project_path, "src");
|
||||
if (fs.existsSync(srcDir)) {
|
||||
const moduleDirs = fs.readdirSync(srcDir, { withFileTypes: true })
|
||||
.filter(d => d.isDirectory() && d.name !== "node_modules")
|
||||
.map(d => d.name);
|
||||
findings.push(`Source modules: ${moduleDirs.join(", ")}`);
|
||||
|
||||
const updatedArch = this.deriveArchitectureFromSource(srcDir, archMd, moduleDirs);
|
||||
if (updatedArch) {
|
||||
ciFiles.writeArchitectureMd(updatedArch);
|
||||
filesUpdated.push(".ciagent/ARCHITECTURE.md");
|
||||
findings.push("Updated ARCHITECTURE.md with source-derived component boundaries");
|
||||
|
||||
decisions.push({
|
||||
id: `D-P${context.phase}-001`,
|
||||
decision: "Updated component boundaries from source scan",
|
||||
rationale: "Source directory structure reveals actual module boundaries",
|
||||
confidence: 0.75,
|
||||
alternatives: ["manual architecture review", "no update"],
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
if (projectMd && archMd) {
|
||||
const updatedProject = this.maybeUpdateKeyDecisions(projectMd, findings);
|
||||
if (updatedProject) {
|
||||
ciFiles.writeProjectMd(updatedProject, "research findings update");
|
||||
filesUpdated.push(".ciagent/PROJECT.md");
|
||||
findings.push("Updated PROJECT.md key decisions from research");
|
||||
|
||||
decisions.push({
|
||||
id: `D-P${context.phase}-002`,
|
||||
decision: "Logged research-based decisions to PROJECT.md",
|
||||
rationale: "Research findings warrant recording as key decisions",
|
||||
confidence: 0.70,
|
||||
alternatives: ["defer decision logging", "log after execution"],
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
this.commitFindings(context, findings, decisions);
|
||||
|
||||
return {
|
||||
success: true,
|
||||
findingsCount: findings.length,
|
||||
decisionsLogged: decisions.length,
|
||||
filesUpdated,
|
||||
};
|
||||
} catch (err) {
|
||||
return {
|
||||
success: false,
|
||||
findingsCount: 0,
|
||||
decisionsLogged: 0,
|
||||
filesUpdated: [],
|
||||
error: `Research failed: ${err instanceof Error ? err.message : String(err)}`,
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
private deriveArchitectureFromSource(srcDir: string, existing: ArchitectureMd | null, moduleDirs: string[]): ArchitectureMd | null {
|
||||
const newComponents = moduleDirs.map(dir => {
|
||||
const dirPath = path.join(srcDir, dir);
|
||||
const fileCount = this.countTsFiles(dirPath);
|
||||
const existingComp = existing?.components.find(c => c.name.toLowerCase() === dir.toLowerCase());
|
||||
|
||||
return {
|
||||
name: existingComp?.name || this.capitalize(dir),
|
||||
description: existingComp?.description || `${dir} module with ${fileCount} source files`,
|
||||
boundaries: existingComp?.boundaries || `src/${dir}/ — ${fileCount} files, internal module`,
|
||||
dependsOn: existingComp?.dependsOn || [],
|
||||
};
|
||||
});
|
||||
|
||||
if (existing) {
|
||||
const existingNames = new Set(existing.components.map(c => c.name.toLowerCase()));
|
||||
const hasNew = newComponents.some(c => !existingNames.has(c.name.toLowerCase()));
|
||||
if (!hasNew) {
|
||||
return {
|
||||
...existing,
|
||||
components: existing.components.map(comp => {
|
||||
const updated = newComponents.find(n => n.name.toLowerCase() === comp.name.toLowerCase());
|
||||
return updated || comp;
|
||||
}),
|
||||
};
|
||||
}
|
||||
|
||||
const merged = [...existing.components];
|
||||
for (const nc of newComponents) {
|
||||
if (!existingNames.has(nc.name.toLowerCase())) {
|
||||
merged.push(nc);
|
||||
}
|
||||
}
|
||||
return { ...existing, components: merged };
|
||||
}
|
||||
|
||||
return {
|
||||
overview: "Architecture derived from source directory scan",
|
||||
components: newComponents,
|
||||
dataFlow: "Modules communicate via typed interfaces and shared utilities",
|
||||
buildOrder: moduleDirs.map(d => `Build ${d} module`),
|
||||
};
|
||||
}
|
||||
|
||||
private maybeUpdateKeyDecisions(projectMd: ProjectMd, findings: string[]): ProjectMd | null {
|
||||
const researchDecisions = findings
|
||||
.filter(f => f.includes("Updated") || f.includes("Found") || f.includes("derived"))
|
||||
.map(f => ({
|
||||
decision: f.slice(0, 50),
|
||||
rationale: "Derived from mechanical source analysis",
|
||||
outcome: "logged by researcher",
|
||||
}));
|
||||
|
||||
if (researchDecisions.length === 0) return null;
|
||||
|
||||
const existingDecisions = projectMd.keyDecisions || [];
|
||||
const existingDecisionTexts = new Set(existingDecisions.map(d => d.decision));
|
||||
|
||||
const novelDecisions = researchDecisions.filter(d => !existingDecisionTexts.has(d.decision));
|
||||
if (novelDecisions.length === 0) return null;
|
||||
|
||||
return {
|
||||
...projectMd,
|
||||
keyDecisions: [...existingDecisions, ...novelDecisions],
|
||||
};
|
||||
}
|
||||
|
||||
private commitFindings(context: AgentContext, findings: string[], decisions: CommitDecision[]): void {
|
||||
try {
|
||||
const gitContext = new GitContext(context.project_path);
|
||||
const projectState = gitContext.reconstructState();
|
||||
const milestone = projectState.currentMilestone || "v1.0";
|
||||
|
||||
const commitMsg = CommitBuilder.buildResearchCommit(
|
||||
context.phase,
|
||||
milestone,
|
||||
`phase ${context.phase} domain research`,
|
||||
findings,
|
||||
decisions.length > 0 ? decisions : undefined,
|
||||
);
|
||||
|
||||
if (fileExists(path.join(context.project_path, ".git"))) {
|
||||
execSync(`git add -A && git commit -m "${commitMsg.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
}
|
||||
} catch (err) {
|
||||
this.warn(`Research commit failed: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
|
||||
private countTsFiles(dir: string): number {
|
||||
if (!fs.existsSync(dir)) return 0;
|
||||
let count = 0;
|
||||
const entries = fs.readdirSync(dir, { withFileTypes: true });
|
||||
for (const entry of entries) {
|
||||
if (entry.isDirectory() && entry.name !== "node_modules") {
|
||||
count += this.countTsFiles(path.join(dir, entry.name));
|
||||
} else if (entry.name.endsWith(".ts") && !entry.name.endsWith(".d.ts") && !entry.name.endsWith(".test.ts")) {
|
||||
count++;
|
||||
}
|
||||
}
|
||||
return count;
|
||||
}
|
||||
|
||||
private capitalize(s: string): string {
|
||||
return s.split("-").map(p => p.charAt(0).toUpperCase() + p.slice(1)).join("-");
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,94 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { TesterAgent } from "../agents/tester.js";
|
||||
import { AgentContext } from "../agents/base.js";
|
||||
import { IntelligenceBackend, BackendRequest, BackendResult } from "../backends/types.js";
|
||||
import { emptyTokenUsage } from "../backends/types.js";
|
||||
|
||||
class MockBackend implements IntelligenceBackend {
|
||||
readonly name = "mock";
|
||||
readonly type = "llm" as const;
|
||||
async isAvailable(): Promise<boolean> { return true; }
|
||||
async execute(request: BackendRequest): Promise<BackendResult> {
|
||||
return {
|
||||
success: true,
|
||||
output: `Mock backend executed: ${request.task.slice(0, 50)}`,
|
||||
artifacts: [],
|
||||
decisions: [],
|
||||
escalations: [],
|
||||
usage: emptyTokenUsage(),
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-tester-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
function makeContext(dir: string, backend?: IntelligenceBackend): AgentContext {
|
||||
return {
|
||||
project_path: dir,
|
||||
phase: 1,
|
||||
stage: "test",
|
||||
specification: "Build a REST API for task management",
|
||||
config_path: path.join(dir, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
}
|
||||
|
||||
describe("TesterAgent", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = createTempDir();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
it("detects test files when src directory exists", async () => {
|
||||
const srcDir = path.join(dir, "src");
|
||||
fs.mkdirSync(srcDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(srcDir, "app.integration.test.ts"), "test('integration', () => {});\n");
|
||||
|
||||
const tester = new TesterAgent();
|
||||
const result = await tester.execute(makeContext(dir));
|
||||
expect(result.success).toBeDefined();
|
||||
});
|
||||
|
||||
it("does not write test files", async () => {
|
||||
const srcDir = path.join(dir, "src");
|
||||
fs.mkdirSync(srcDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(srcDir, "app.test.ts"), "test('unit', () => {});\n");
|
||||
|
||||
const testFilesBefore = fs.readdirSync(srcDir).filter(f => f.endsWith(".test.ts"));
|
||||
const tester = new TesterAgent();
|
||||
await tester.execute(makeContext(dir));
|
||||
const testFilesAfter = fs.readdirSync(srcDir).filter(f => f.endsWith(".test.ts"));
|
||||
expect(testFilesAfter.length).toBe(testFilesBefore.length);
|
||||
});
|
||||
|
||||
it("delegates to backend when available", async () => {
|
||||
const mockBackend = new MockBackend();
|
||||
const tester = new TesterAgent();
|
||||
const result = await tester.execute(makeContext(dir, mockBackend));
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("Mock backend executed");
|
||||
});
|
||||
|
||||
it("has correct agent name", () => {
|
||||
const tester = new TesterAgent();
|
||||
expect(tester.name).toBe("tester");
|
||||
});
|
||||
|
||||
it("has correct workflow", () => {
|
||||
const tester = new TesterAgent();
|
||||
expect(tester.workflow).toBe("test");
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,181 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { BaseAgent, AgentContext, AgentResult } from "./base.js";
|
||||
import { execSync } from "node:child_process";
|
||||
|
||||
export interface TesterResult {
|
||||
success: boolean;
|
||||
integrationTestsFound: number;
|
||||
integrationTestsPassed: number;
|
||||
e2eTestsFound: number;
|
||||
e2eTestsPassed: number;
|
||||
overallPassed: boolean;
|
||||
error?: string;
|
||||
}
|
||||
|
||||
export class TesterAgent extends BaseAgent {
|
||||
readonly name = "tester";
|
||||
readonly description = "Runs integration, e2e, functional tests. Validates non-unit test coverage.";
|
||||
readonly workflow = "test";
|
||||
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
const start = Date.now();
|
||||
this.log("Running automated tests...");
|
||||
|
||||
if (context.backend) {
|
||||
const result = await this.executeViaBackend(
|
||||
context,
|
||||
`Run integration, e2e, and functional tests for phase ${context.phase}. Specification: ${context.specification}. Detect *.integration.test.ts, *.e2e.test.ts, *.functional.test.ts files. Run npm test. Parse output for pass/fail counts per category. Report structured TesterResult. Do NOT write any test files — only detect and run existing ones.`
|
||||
);
|
||||
return { ...result, duration_ms: Date.now() - start };
|
||||
}
|
||||
|
||||
const result = await this.runMechanicalTests(context);
|
||||
const output = JSON.stringify(result, null, 2);
|
||||
|
||||
return {
|
||||
success: result.success,
|
||||
output,
|
||||
artifacts_created: [],
|
||||
decisions: 0,
|
||||
escalations: result.overallPassed ? 0 : 1,
|
||||
duration_ms: Date.now() - start,
|
||||
error: result.error,
|
||||
};
|
||||
}
|
||||
|
||||
private async runMechanicalTests(context: AgentContext): Promise<TesterResult> {
|
||||
try {
|
||||
const srcDir = path.join(context.project_path, "src");
|
||||
const integrationFiles = fs.existsSync(srcDir) ? this.findTestFiles(srcDir, /\.integration\.test\.ts$/) : [];
|
||||
const e2eFiles = fs.existsSync(srcDir) ? this.findTestFiles(srcDir, /\.e2e\.test\.ts$/) : [];
|
||||
const functionalFiles = fs.existsSync(srcDir) ? this.findTestFiles(srcDir, /\.functional\.test\.ts$/) : [];
|
||||
|
||||
const integrationTestsFound = integrationFiles.length;
|
||||
const e2eTestsFound = e2eFiles.length + functionalFiles.length;
|
||||
|
||||
let overallPassed = false;
|
||||
let integrationTestsPassed = 0;
|
||||
let e2eTestsPassed = 0;
|
||||
|
||||
try {
|
||||
const testOutput = execSync("npm test 2>&1", {
|
||||
cwd: context.project_path,
|
||||
encoding: "utf-8",
|
||||
stdio: ["pipe", "pipe", "pipe"],
|
||||
timeout: 120000,
|
||||
});
|
||||
overallPassed = true;
|
||||
|
||||
const passCounts = this.parseTestOutput(testOutput);
|
||||
integrationTestsPassed = integrationTestsFound > 0 ? integrationTestsFound : 0;
|
||||
e2eTestsPassed = e2eTestsFound > 0 ? e2eTestsFound : 0;
|
||||
|
||||
if (integrationTestsFound > 0) {
|
||||
integrationTestsPassed = this.estimateCategoryPassed(testOutput, "integration");
|
||||
}
|
||||
if (e2eTestsFound > 0) {
|
||||
e2eTestsPassed = this.estimateCategoryPassed(testOutput, "e2e");
|
||||
}
|
||||
} catch (err) {
|
||||
const output = err instanceof Error && "stdout" in err
|
||||
? (err as unknown as { stdout: string }).stdout || ""
|
||||
: "";
|
||||
const stderr = err instanceof Error && "stderr" in err
|
||||
? (err as unknown as { stderr: string }).stderr || ""
|
||||
: "";
|
||||
|
||||
const combined = `${output}\n${stderr}`;
|
||||
overallPassed = false;
|
||||
|
||||
const passCounts = this.parseTestOutput(combined);
|
||||
|
||||
if (integrationTestsFound > 0) {
|
||||
integrationTestsPassed = this.estimateCategoryPassed(combined, "integration");
|
||||
}
|
||||
if (e2eTestsFound > 0) {
|
||||
e2eTestsPassed = this.estimateCategoryPassed(combined, "e2e");
|
||||
}
|
||||
|
||||
return {
|
||||
success: false,
|
||||
integrationTestsFound,
|
||||
integrationTestsPassed,
|
||||
e2eTestsFound,
|
||||
e2eTestsPassed,
|
||||
overallPassed: false,
|
||||
error: `npm test failed: ${err instanceof Error ? err.message : String(err)}`,
|
||||
};
|
||||
}
|
||||
|
||||
return {
|
||||
success: overallPassed,
|
||||
integrationTestsFound,
|
||||
integrationTestsPassed,
|
||||
e2eTestsFound,
|
||||
e2eTestsPassed,
|
||||
overallPassed,
|
||||
};
|
||||
} catch (err) {
|
||||
return {
|
||||
success: false,
|
||||
integrationTestsFound: 0,
|
||||
integrationTestsPassed: 0,
|
||||
e2eTestsFound: 0,
|
||||
e2eTestsPassed: 0,
|
||||
overallPassed: false,
|
||||
error: `Test execution failed: ${err instanceof Error ? err.message : String(err)}`,
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
private findTestFiles(dir: string, pattern: RegExp): string[] {
|
||||
const files: string[] = [];
|
||||
if (!fs.existsSync(dir)) return files;
|
||||
const entries = fs.readdirSync(dir, { withFileTypes: true });
|
||||
for (const entry of entries) {
|
||||
const fullPath = path.join(dir, entry.name);
|
||||
if (entry.isDirectory() && entry.name !== "node_modules") {
|
||||
files.push(...this.findTestFiles(fullPath, pattern));
|
||||
} else if (pattern.test(entry.name)) {
|
||||
files.push(fullPath);
|
||||
}
|
||||
}
|
||||
return files;
|
||||
}
|
||||
|
||||
private parseTestOutput(output: string): { total: number; passed: number; failed: number } {
|
||||
const jestSummary = output.match(/Tests:\s+(\d+)\s+passed(?:,\s+(\d+)\s+failed)?/);
|
||||
if (jestSummary) {
|
||||
const passed = parseInt(jestSummary[1], 10) || 0;
|
||||
const failed = parseInt(jestSummary[2], 10) || 0;
|
||||
return { total: passed + failed, passed, failed };
|
||||
}
|
||||
|
||||
const jestAlt = output.match(/(\d+)\s+passing/);
|
||||
const jestAltFail = output.match(/(\d+)\s+failing/);
|
||||
if (jestAlt) {
|
||||
const passed = parseInt(jestAlt[1], 10) || 0;
|
||||
const failed = jestAltFail ? parseInt(jestAltFail[1], 10) || 0 : 0;
|
||||
return { total: passed + failed, passed, failed };
|
||||
}
|
||||
|
||||
return { total: 0, passed: 0, failed: 0 };
|
||||
}
|
||||
|
||||
private estimateCategoryPassed(output: string, category: string): number {
|
||||
const categoryPattern = category === "integration"
|
||||
? /\.integration\.test\.ts/g
|
||||
: /\.e2e\.test\.ts|\.functional\.test\.ts/g;
|
||||
|
||||
const mentions = (output.match(categoryPattern) || []).length;
|
||||
if (mentions > 0) {
|
||||
const failPattern = /FAIL|failed|error/i;
|
||||
const lines = output.split("\n").filter(l => categoryPattern.test(l));
|
||||
const failed = lines.filter(l => failPattern.test(l)).length;
|
||||
return Math.max(mentions - failed, 0);
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,100 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { VerifierAgent } from "../agents/verifier.js";
|
||||
import { AgentContext } from "../agents/base.js";
|
||||
import { IntelligenceBackend, BackendRequest, BackendResult } from "../backends/types.js";
|
||||
import { emptyTokenUsage } from "../backends/types.js";
|
||||
|
||||
class MockBackend implements IntelligenceBackend {
|
||||
readonly name = "mock";
|
||||
readonly type = "llm" as const;
|
||||
async isAvailable(): Promise<boolean> { return true; }
|
||||
async execute(request: BackendRequest): Promise<BackendResult> {
|
||||
return {
|
||||
success: true,
|
||||
output: `Mock backend executed: ${request.task.slice(0, 50)}`,
|
||||
artifacts: [],
|
||||
decisions: [],
|
||||
escalations: [],
|
||||
usage: emptyTokenUsage(),
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-verifier-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
function makeContext(dir: string, backend?: IntelligenceBackend): AgentContext {
|
||||
return {
|
||||
project_path: dir,
|
||||
phase: 1,
|
||||
stage: "verify",
|
||||
specification: "Build a REST API for task management",
|
||||
config_path: path.join(dir, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
}
|
||||
|
||||
function setupBasicProject(dir: string): void {
|
||||
const ciDir = path.join(dir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), "{}");
|
||||
|
||||
const srcDir = path.join(dir, "src");
|
||||
fs.mkdirSync(srcDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(srcDir, "index.ts"), 'export const VERSION = "0.7.0";\n');
|
||||
}
|
||||
|
||||
describe("VerifierAgent", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = createTempDir();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
it("runs mechanical verification without backend", async () => {
|
||||
setupBasicProject(dir);
|
||||
const verifier = new VerifierAgent();
|
||||
const result = await verifier.execute(makeContext(dir));
|
||||
expect(result.output).toBeDefined();
|
||||
});
|
||||
|
||||
it("is read-only — does not create new source files", async () => {
|
||||
setupBasicProject(dir);
|
||||
const srcDir = path.join(dir, "src");
|
||||
const filesBefore = fs.readdirSync(srcDir);
|
||||
const verifier = new VerifierAgent();
|
||||
await verifier.execute(makeContext(dir));
|
||||
const filesAfter = fs.readdirSync(srcDir);
|
||||
expect(filesAfter.length).toBe(filesBefore.length);
|
||||
});
|
||||
|
||||
it("delegates to backend when available", async () => {
|
||||
setupBasicProject(dir);
|
||||
const mockBackend = new MockBackend();
|
||||
const verifier = new VerifierAgent();
|
||||
const result = await verifier.execute(makeContext(dir, mockBackend));
|
||||
expect(result.success).toBe(true);
|
||||
expect(result.output).toContain("Mock backend executed");
|
||||
});
|
||||
|
||||
it("has correct agent name", () => {
|
||||
const verifier = new VerifierAgent();
|
||||
expect(verifier.name).toBe("verifier");
|
||||
});
|
||||
|
||||
it("has correct workflow", () => {
|
||||
const verifier = new VerifierAgent();
|
||||
expect(verifier.workflow).toBe("verify");
|
||||
});
|
||||
});
|
||||
+217
-5
@@ -1,4 +1,22 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { BaseAgent, AgentContext, AgentResult } from "./base.js";
|
||||
import { VerificationPipeline } from "../verification/index.js";
|
||||
import { CommitBuilder, VerifyCommitInput } from "../core/commit-builder.js";
|
||||
import { GitContext } from "../core/git-context.js";
|
||||
import { CIAgentFiles } from "../core/ciagent-files.js";
|
||||
import { fileExists } from "../utils/file.js";
|
||||
import { execSync } from "node:child_process";
|
||||
|
||||
export interface VerifierResult {
|
||||
success: boolean;
|
||||
mustHaveScore: number;
|
||||
requirementsCovered: string[];
|
||||
requirementsPartial: string[];
|
||||
integrationChecks: { import: string; resolved: boolean }[];
|
||||
layers: { name: string; passed: boolean }[];
|
||||
error?: string;
|
||||
}
|
||||
|
||||
export class VerifierAgent extends BaseAgent {
|
||||
readonly name = "verifier";
|
||||
@@ -8,21 +26,215 @@ export class VerifierAgent extends BaseAgent {
|
||||
async execute(context: AgentContext): Promise<AgentResult> {
|
||||
const start = Date.now();
|
||||
this.log("Verifying phase output...");
|
||||
|
||||
if (context.backend) {
|
||||
const result = await this.executeViaBackend(
|
||||
context,
|
||||
`Verify phase ${context.phase} output. Specification: ${context.specification}`
|
||||
`Verify phase ${context.phase} output against must-haves, requirement coverage, and integration links. Specification: ${context.specification}. Check all .ciagent/ reference files. Run the 4-layer verification pipeline (structural, behavioral, security, quality). Verify imports resolve. Report structured VerifierResult.`
|
||||
);
|
||||
return { ...result, duration_ms: Date.now() - start };
|
||||
}
|
||||
|
||||
const result = await this.runMechanicalVerification(context);
|
||||
const output = JSON.stringify(result, null, 2);
|
||||
|
||||
return {
|
||||
success: false,
|
||||
output: "Verification requires an intelligence backend. Configure one with: ci init --backend",
|
||||
success: result.success,
|
||||
output,
|
||||
artifacts_created: [],
|
||||
decisions: 0,
|
||||
escalations: 0,
|
||||
escalations: result.success ? 0 : 1,
|
||||
duration_ms: Date.now() - start,
|
||||
error: "No intelligence backend available",
|
||||
error: result.error,
|
||||
};
|
||||
}
|
||||
|
||||
private async runMechanicalVerification(context: AgentContext): Promise<VerifierResult> {
|
||||
try {
|
||||
const pipeline = new VerificationPipeline(context.project_path);
|
||||
const pipelineResult = await pipeline.run(context.phase);
|
||||
|
||||
const layers: { name: string; passed: boolean }[] = [
|
||||
{ name: pipelineResult.structural.name, passed: pipelineResult.structural.passed },
|
||||
{ name: pipelineResult.behavioral.name, passed: pipelineResult.behavioral.passed },
|
||||
{ name: pipelineResult.security.name, passed: pipelineResult.security.passed },
|
||||
{ name: pipelineResult.quality.name, passed: pipelineResult.quality.passed },
|
||||
];
|
||||
|
||||
const gitContext = new GitContext(context.project_path);
|
||||
const ciFiles = new CIAgentFiles(context.project_path);
|
||||
|
||||
const mustHaveScore = this.checkMustHaves(context, gitContext, ciFiles);
|
||||
const reqCoverage = this.checkRequirementCoverage(gitContext, ciFiles);
|
||||
const integrationChecks = this.checkIntegrationLinks(context.project_path);
|
||||
|
||||
const allPassed = pipelineResult.all_passed &&
|
||||
mustHaveScore >= 1.0 &&
|
||||
reqCoverage.partial.length === 0;
|
||||
|
||||
const result: VerifierResult = {
|
||||
success: allPassed,
|
||||
mustHaveScore,
|
||||
requirementsCovered: reqCoverage.covered,
|
||||
requirementsPartial: reqCoverage.partial,
|
||||
integrationChecks,
|
||||
layers,
|
||||
};
|
||||
|
||||
if (!allPassed) {
|
||||
result.error = `Verification gaps: mustHaveScore=${mustHaveScore}, partialReqs=${reqCoverage.partial.join(",")}, layerFailures=${layers.filter(l => !l.passed).map(l => l.name).join(",")}`;
|
||||
}
|
||||
|
||||
this.commitVerificationResult(context, result, ciFiles);
|
||||
|
||||
return result;
|
||||
} catch (err) {
|
||||
return {
|
||||
success: false,
|
||||
mustHaveScore: 0,
|
||||
requirementsCovered: [],
|
||||
requirementsPartial: [],
|
||||
integrationChecks: [],
|
||||
layers: [],
|
||||
error: `Verification failed: ${err instanceof Error ? err.message : String(err)}`,
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
private checkMustHaves(context: AgentContext, gitContext: GitContext, ciFiles: CIAgentFiles): number {
|
||||
const roadmap = ciFiles.readRoadmapMd();
|
||||
if (!roadmap) return 0;
|
||||
|
||||
const currentPhase = roadmap.phases.find(p => p.number === context.phase);
|
||||
if (!currentPhase) return 0;
|
||||
|
||||
const successCriteria = currentPhase.successCriteria;
|
||||
if (successCriteria.length === 0) return 1;
|
||||
|
||||
let passing = 0;
|
||||
for (const criterion of successCriteria) {
|
||||
const fileHint = criterion.match(/(?:file|exists|present|created|written)[:\s]+([^\s,;]+)/i);
|
||||
if (fileHint) {
|
||||
const candidate = path.join(context.project_path, fileHint[1]);
|
||||
if (fileExists(candidate)) {
|
||||
passing++;
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
if (fileExists(path.join(context.project_path, ".ciagent"))) {
|
||||
passing++;
|
||||
}
|
||||
}
|
||||
|
||||
return Math.min(passing / successCriteria.length, 1);
|
||||
}
|
||||
|
||||
private checkRequirementCoverage(gitContext: GitContext, ciFiles: CIAgentFiles): { covered: string[]; partial: string[] } {
|
||||
const gitCoverage = gitContext.getRequirementsCoverage();
|
||||
const reqsMd = ciFiles.readRequirementsMd();
|
||||
|
||||
if (!reqsMd || reqsMd.traceability.length === 0) {
|
||||
return { covered: gitCoverage.covered, partial: gitCoverage.partial };
|
||||
}
|
||||
|
||||
const covered = new Set(gitCoverage.covered);
|
||||
const partial = new Set(gitCoverage.partial);
|
||||
|
||||
for (const t of reqsMd.traceability) {
|
||||
if (t.status === "complete") {
|
||||
covered.add(t.requirement);
|
||||
partial.delete(t.requirement);
|
||||
} else if (t.status === "in_progress" || t.status === "blocked") {
|
||||
partial.add(t.requirement);
|
||||
}
|
||||
}
|
||||
|
||||
return {
|
||||
covered: [...covered].sort(),
|
||||
partial: [...partial].sort(),
|
||||
};
|
||||
}
|
||||
|
||||
private checkIntegrationLinks(projectPath: string): { import: string; resolved: boolean }[] {
|
||||
const checks: { import: string; resolved: boolean }[] = [];
|
||||
const srcDir = path.join(projectPath, "src");
|
||||
|
||||
if (!fs.existsSync(srcDir)) return checks;
|
||||
|
||||
const tsFiles = this.collectTsFiles(srcDir);
|
||||
const importPattern = /import\s+.*from\s+['"](\.\/[^'"]+)['"]/g;
|
||||
|
||||
for (const file of tsFiles) {
|
||||
const content = fs.readFileSync(file, "utf-8");
|
||||
let match: RegExpExecArray | null;
|
||||
while ((match = importPattern.exec(content)) !== null) {
|
||||
const importPath = match[1];
|
||||
const resolved = this.resolveImport(file, importPath);
|
||||
if (importPath.startsWith(".")) {
|
||||
checks.push({ import: `${path.relative(projectPath, file)}:${importPath}`, resolved: resolved !== null });
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return checks;
|
||||
}
|
||||
|
||||
private commitVerificationResult(context: AgentContext, result: VerifierResult, ciFiles: CIAgentFiles): void {
|
||||
try {
|
||||
const projectState = new GitContext(context.project_path).reconstructState();
|
||||
const milestone = projectState.currentMilestone || "v1.0";
|
||||
|
||||
const verifyInput: VerifyCommitInput = {
|
||||
phase: context.phase,
|
||||
milestone,
|
||||
subject: result.success ? "passed" : "gaps_found",
|
||||
requirements: {
|
||||
covered: result.requirementsCovered,
|
||||
partial: result.requirementsPartial,
|
||||
},
|
||||
lessons: result.success ? ["All verification checks passed"] : [`Must-have score: ${result.mustHaveScore}`, `Layer failures: ${result.layers.filter(l => !l.passed).map(l => l.name).join(", ")}`],
|
||||
};
|
||||
|
||||
const commitMsg = CommitBuilder.buildVerifyCommit(verifyInput);
|
||||
if (fileExists(path.join(context.project_path, ".git"))) {
|
||||
execSync(`git add -A && git commit -m "${commitMsg.replace(/"/g, '\\"')}" --allow-empty`, {
|
||||
cwd: context.project_path,
|
||||
stdio: "pipe",
|
||||
});
|
||||
}
|
||||
} catch (err) {
|
||||
this.warn(`Verification commit failed: ${err instanceof Error ? err.message : String(err)}`);
|
||||
}
|
||||
}
|
||||
|
||||
private collectTsFiles(dir: string): string[] {
|
||||
const files: string[] = [];
|
||||
if (!fs.existsSync(dir)) return files;
|
||||
const entries = fs.readdirSync(dir, { withFileTypes: true });
|
||||
for (const entry of entries) {
|
||||
const fullPath = path.join(dir, entry.name);
|
||||
if (entry.isDirectory() && entry.name !== "node_modules") {
|
||||
files.push(...this.collectTsFiles(fullPath));
|
||||
} else if (entry.name.endsWith(".ts") && !entry.name.endsWith(".d.ts") && !entry.name.endsWith(".test.ts")) {
|
||||
files.push(fullPath);
|
||||
}
|
||||
}
|
||||
return files;
|
||||
}
|
||||
|
||||
private resolveImport(fromFile: string, importPath: string): string | null {
|
||||
if (!importPath.startsWith(".")) return null;
|
||||
const dir = path.dirname(fromFile);
|
||||
const candidates = [
|
||||
path.resolve(dir, importPath + ".ts"),
|
||||
path.resolve(dir, importPath + ".js"),
|
||||
path.resolve(dir, importPath, "index.ts"),
|
||||
path.resolve(dir, importPath, "index.js"),
|
||||
];
|
||||
for (const candidate of candidates) {
|
||||
if (fs.existsSync(candidate)) return candidate;
|
||||
}
|
||||
return null;
|
||||
}
|
||||
}
|
||||
@@ -42,7 +42,7 @@ describe("OllamaBaseBackend", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-ollama-base-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-ollama-base-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
|
||||
@@ -15,16 +15,26 @@ import {
|
||||
import { AgentName, ModelProfile } from "../types/config.js";
|
||||
import { Decision } from "../types/decisions.js";
|
||||
import { Escalation } from "../types/escalation.js";
|
||||
import { ToolRegistry, ToolCall, ToolResult } from "./tool-registry.js";
|
||||
import { ToolRegistry, ToolCall, ToolResult, ToolDefinition } from "./tool-registry.js";
|
||||
|
||||
const MAX_TOOL_ROUNDS = 50;
|
||||
|
||||
const PERSONA_TOOL_MAP: Record<string, string> = {
|
||||
read: "readFile",
|
||||
write: "writeFile",
|
||||
edit: "editFile",
|
||||
bash: "runBash",
|
||||
glob: "glob",
|
||||
grep: "grep",
|
||||
};
|
||||
|
||||
export abstract class OllamaBaseBackend implements IntelligenceBackend {
|
||||
abstract readonly name: string;
|
||||
readonly type: BackendType = "llm";
|
||||
|
||||
protected config: LLMBackendConfig;
|
||||
protected projectPath: string;
|
||||
protected filteredToolSchema: Array<Record<string, unknown>> | null = null;
|
||||
|
||||
constructor(config: LLMBackendConfig | undefined) {
|
||||
this.config = config || { base_url: "http://localhost:11434", model_profile: "balanced" };
|
||||
@@ -42,6 +52,9 @@ export abstract class OllamaBaseBackend implements IntelligenceBackend {
|
||||
const model = this.resolveModel();
|
||||
|
||||
const toolRegistry = new ToolRegistry(request.context.project_path);
|
||||
const allowedTools = this.parsePersonaTools(personaContent);
|
||||
const filteredDefinitions = this.filterToolDefinitions(toolRegistry.getDefinitions(), allowedTools);
|
||||
this.filteredToolSchema = this.definitionsToOpenAISchema(filteredDefinitions);
|
||||
|
||||
const messages: OllamaMessage[] = [];
|
||||
messages.push({
|
||||
@@ -62,7 +75,7 @@ export abstract class OllamaBaseBackend implements IntelligenceBackend {
|
||||
|
||||
while (round < MAX_TOOL_ROUNDS) {
|
||||
round++;
|
||||
const response = await this.callModel(messages, model, toolRegistry);
|
||||
const response = await this.callModelWithTools(messages, model, filteredDefinitions);
|
||||
|
||||
totalInputTokens += response.usage?.prompt_tokens || 0;
|
||||
totalOutputTokens += response.usage?.completion_tokens || 0;
|
||||
@@ -124,6 +137,65 @@ export abstract class OllamaBaseBackend implements IntelligenceBackend {
|
||||
}
|
||||
}
|
||||
|
||||
protected parsePersonaTools(personaContent: string): string[] | null {
|
||||
const frontmatterMatch = personaContent.match(/^---\n([\s\S]*?)\n---/);
|
||||
if (!frontmatterMatch) return null;
|
||||
|
||||
const frontmatter = frontmatterMatch[1];
|
||||
const toolsMatch = frontmatter.match(/tools:\s*\n((?:\s+\w+:.+\n?)+)/);
|
||||
if (!toolsMatch) {
|
||||
const inlineMatch = frontmatter.match(/tools:\s*\[([^\]]+)\]/);
|
||||
if (inlineMatch) {
|
||||
return inlineMatch[1]
|
||||
.split(",")
|
||||
.map((t) => t.trim())
|
||||
.filter(Boolean)
|
||||
.map((t) => PERSONA_TOOL_MAP[t] || t);
|
||||
}
|
||||
return null;
|
||||
}
|
||||
|
||||
const toolsBlock = toolsMatch[1];
|
||||
const toolNames: string[] = [];
|
||||
const lineRegex = /^\s+(\w+):/gm;
|
||||
let lineMatch;
|
||||
while ((lineMatch = lineRegex.exec(toolsBlock)) !== null) {
|
||||
const personaToolName = lineMatch[1];
|
||||
toolNames.push(PERSONA_TOOL_MAP[personaToolName] || personaToolName);
|
||||
}
|
||||
|
||||
return toolNames.length > 0 ? toolNames : null;
|
||||
}
|
||||
|
||||
protected filterToolDefinitions(definitions: ToolDefinition[], allowedTools: string[] | null): ToolDefinition[] {
|
||||
if (!allowedTools) return definitions;
|
||||
const allowedSet = new Set(allowedTools);
|
||||
return definitions.filter((def) => allowedSet.has(def.name));
|
||||
}
|
||||
|
||||
protected async callModelWithTools(
|
||||
messages: OllamaMessage[],
|
||||
model: string,
|
||||
toolDefinitions: ToolDefinition[]
|
||||
): Promise<OllamaChatResponse> {
|
||||
return this.callModel(messages, model, new ToolRegistry(this.projectPath));
|
||||
}
|
||||
|
||||
protected definitionsToOpenAISchema(definitions: ToolDefinition[]): Array<Record<string, unknown>> {
|
||||
return definitions.map((def) => ({
|
||||
type: "function",
|
||||
function: {
|
||||
name: def.name,
|
||||
description: def.description,
|
||||
parameters: def.parameters,
|
||||
},
|
||||
}));
|
||||
}
|
||||
|
||||
protected getActiveToolSchema(toolRegistry: ToolRegistry): Array<Record<string, unknown>> {
|
||||
return this.filteredToolSchema || toolRegistry.getOpenAIToolSchema();
|
||||
}
|
||||
|
||||
protected abstract callModel(
|
||||
messages: OllamaMessage[],
|
||||
model: string,
|
||||
@@ -170,7 +242,7 @@ export abstract class OllamaBaseBackend implements IntelligenceBackend {
|
||||
return fs.readFileSync(candidate, "utf-8");
|
||||
}
|
||||
}
|
||||
return `You are the CI ${persona} agent. Execute the requested task thoroughly and autonomously.`;
|
||||
return `You are the CIAgent ${persona} agent. Execute the requested task thoroughly and autonomously.`;
|
||||
}
|
||||
|
||||
protected loadWorkflow(workflow: string): string {
|
||||
|
||||
@@ -61,7 +61,7 @@ export class OllamaCloudBackend extends OllamaBaseBackend {
|
||||
if (m.tool_calls) msg.tool_calls = m.tool_calls;
|
||||
return msg;
|
||||
}),
|
||||
tools: toolRegistry.getOpenAIToolSchema(),
|
||||
tools: this.getActiveToolSchema(toolRegistry),
|
||||
stream: false,
|
||||
};
|
||||
|
||||
|
||||
@@ -48,7 +48,7 @@ export class OllamaLocalBackend extends OllamaBaseBackend {
|
||||
if (m.tool_calls) msg.tool_calls = m.tool_calls;
|
||||
return msg;
|
||||
}),
|
||||
tools: toolRegistry.getOpenAIToolSchema(),
|
||||
tools: this.getActiveToolSchema(toolRegistry),
|
||||
stream: false,
|
||||
};
|
||||
|
||||
|
||||
@@ -8,7 +8,7 @@ describe("ToolRegistry Extended", () => {
|
||||
let registry: ToolRegistry;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-tool-registry-ext-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-tool-registry-ext-"));
|
||||
registry = new ToolRegistry(tempDir);
|
||||
});
|
||||
|
||||
|
||||
@@ -8,7 +8,7 @@ describe("ToolRegistry", () => {
|
||||
let registry: ToolRegistry;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-tool-registry-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-tool-registry-test-"));
|
||||
registry = new ToolRegistry(tempDir);
|
||||
});
|
||||
|
||||
|
||||
+174
-50
@@ -1,6 +1,6 @@
|
||||
import { Command } from "commander";
|
||||
import { CIConfig, AutonomyLevel } from "../types/config.js";
|
||||
import { initCI, loadConfig, isCIInitialized, saveConfig } from "../core/config.js";
|
||||
import { CIAgentConfig, AutonomyLevel } from "../types/config.js";
|
||||
import { initCIAgent, loadConfig, isCIAgentInitialized, saveConfig } from "../core/config.js";
|
||||
import { Specification, parseSpecification } from "../types/specification.js";
|
||||
import { saveSpecification } from "../core/clarify.js";
|
||||
import { OrchestratorAgent } from "../agents/orchestrator.js";
|
||||
@@ -15,13 +15,15 @@ import { PipelineState, createInitialPipelineState } from "../types/pipeline.js"
|
||||
import { resolveBackend } from "../backends/index.js";
|
||||
import { BackendUnavailableError } from "../backends/types.js";
|
||||
import { getAgent } from "../agents/index.js";
|
||||
import { CIAgentFiles } from "../core/ciagent-files.js";
|
||||
import { GiteaClient, generateReleaseNotes } from "../core/gitea.js";
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { execSync } from "node:child_process";
|
||||
|
||||
export function createInitCommand(): Command {
|
||||
return new Command("init")
|
||||
.description("Initialize a new CI project from a specification")
|
||||
.description("Initialize a new CIAgent project from a specification")
|
||||
.argument("[specification]", "Inline specification text")
|
||||
.option("-s, --spec <file>", "Specification file path")
|
||||
.option("-c, --clarify", "Start interactive clarify phase", false)
|
||||
@@ -36,9 +38,9 @@ export function createInitCommand(): Command {
|
||||
.action(async (specification, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (isCIInitialized(projectPath)) {
|
||||
console.log("CI project already initialized in this directory.");
|
||||
console.log("Use 'ci run' to execute the pipeline or 'ci status' to check progress.");
|
||||
if (isCIAgentInitialized(projectPath)) {
|
||||
console.log("CIAgent project already initialized in this directory.");
|
||||
console.log("Use 'ciagent run' to execute the pipeline or 'ciagent status' to check progress.");
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -60,7 +62,7 @@ export function createInitCommand(): Command {
|
||||
}
|
||||
|
||||
const autonomyLevel = options.autonomy as AutonomyLevel;
|
||||
const config: Partial<CIConfig> = {
|
||||
const config: Partial<CIAgentConfig> = {
|
||||
autonomy: {
|
||||
level: autonomyLevel,
|
||||
escalation_hooks: ["deploy", "delete_data", "merge_to_main"],
|
||||
@@ -86,8 +88,8 @@ export function createInitCommand(): Command {
|
||||
},
|
||||
};
|
||||
|
||||
const fullConfig = initCI(projectPath, config);
|
||||
console.log(`✓ CI project initialized (autonomy: ${autonomyLevel})`);
|
||||
const fullConfig = initCIAgent(projectPath, config);
|
||||
console.log(`✓ CIAgent project initialized (autonomy: ${autonomyLevel})`);
|
||||
console.log(` Backend: ${options.backend || "auto"}`);
|
||||
|
||||
if (specText) {
|
||||
@@ -115,15 +117,15 @@ export function createInitCommand(): Command {
|
||||
}
|
||||
}
|
||||
|
||||
console.log("\nConfiguration saved to .ci/config.json");
|
||||
console.log("\nConfiguration saved to .ciagent/config.json");
|
||||
console.log("\nNext steps:");
|
||||
console.log(" ci run --all # Run full pipeline");
|
||||
console.log(" ci run research # Run specific phase");
|
||||
console.log(" ci status # Check project status");
|
||||
console.log(" ciagent run --all # Run full pipeline");
|
||||
console.log(" ciagent run research # Run specific phase");
|
||||
console.log(" ciagent status # Check project status");
|
||||
});
|
||||
}
|
||||
|
||||
async function resolveBackendForCommand(config: CIConfig, overrideBackend?: string): Promise<{ backend: import("../backends/types.js").IntelligenceBackend | undefined; error?: string }> {
|
||||
async function resolveBackendForCommand(config: CIAgentConfig, overrideBackend?: string): Promise<{ backend: import("../backends/types.js").IntelligenceBackend | undefined; error?: string }> {
|
||||
const backendConfig = { ...config.backend };
|
||||
if (overrideBackend) {
|
||||
backendConfig.provider = overrideBackend as typeof backendConfig.provider;
|
||||
@@ -168,8 +170,8 @@ export function createRunCommand(): Command {
|
||||
.action(async (phase, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -187,7 +189,7 @@ export function createRunCommand(): Command {
|
||||
phase: parseInt(options.phase) || 1,
|
||||
stage: phase || "all",
|
||||
specification: "",
|
||||
config_path: path.join(projectPath, ".ci", "config.json"),
|
||||
config_path: path.join(projectPath, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
|
||||
@@ -196,7 +198,7 @@ export function createRunCommand(): Command {
|
||||
context.specification = spec.raw_content;
|
||||
}
|
||||
|
||||
console.log(`Running CI pipeline...`);
|
||||
console.log(`Running CIAgent pipeline...`);
|
||||
if (options.all) {
|
||||
console.log(" Mode: Full pipeline (all phases)");
|
||||
} else {
|
||||
@@ -226,16 +228,16 @@ export function createQuickCommand(): Command {
|
||||
const projectPath = process.cwd();
|
||||
console.log(`Quick task: ${description}`);
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
const config = initCI(projectPath);
|
||||
console.log("Initialized temporary CI project");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
const config = initCIAgent(projectPath);
|
||||
console.log("Initialized temporary CIAgent project");
|
||||
}
|
||||
|
||||
const config = loadConfig(projectPath);
|
||||
const { backend, error: backendError } = await resolveBackendForCommand(config, options.backend);
|
||||
|
||||
if (!backend) {
|
||||
console.error(`\n✗ "ci quick" requires an intelligence backend.`);
|
||||
console.error(`\n✗ "ciagent quick" requires an intelligence backend.`);
|
||||
if (backendError) console.error(` ${backendError}`);
|
||||
process.exit(1);
|
||||
}
|
||||
@@ -249,7 +251,7 @@ export function createQuickCommand(): Command {
|
||||
phase: 0,
|
||||
stage: "all",
|
||||
specification: description,
|
||||
config_path: path.join(projectPath, ".ci", "config.json"),
|
||||
config_path: path.join(projectPath, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
|
||||
@@ -274,8 +276,8 @@ export function createDebugCommand(): Command {
|
||||
.action(async (description, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -283,7 +285,7 @@ export function createDebugCommand(): Command {
|
||||
const { backend, error: backendError } = await resolveBackendForCommand(config, options.backend);
|
||||
|
||||
if (!backend) {
|
||||
console.error(`\n✗ "ci debug" requires an intelligence backend.`);
|
||||
console.error(`\n✗ "ciagent debug" requires an intelligence backend.`);
|
||||
if (backendError) console.error(` ${backendError}`);
|
||||
process.exit(1);
|
||||
}
|
||||
@@ -300,7 +302,7 @@ export function createDebugCommand(): Command {
|
||||
phase: 0,
|
||||
stage: "debug",
|
||||
specification: description || "",
|
||||
config_path: path.join(projectPath, ".ci", "config.json"),
|
||||
config_path: path.join(projectPath, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
|
||||
@@ -324,8 +326,8 @@ export function createVerifyCommand(): Command {
|
||||
.action(async (phase, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -371,8 +373,8 @@ export function createReviewCommand(): Command {
|
||||
.action(async (phase, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -380,7 +382,7 @@ export function createReviewCommand(): Command {
|
||||
const { backend, error: backendError } = await resolveBackendForCommand(config, options.backend);
|
||||
|
||||
if (!backend) {
|
||||
console.error(`\n✗ "ci review" requires an intelligence backend.`);
|
||||
console.error(`\n✗ "ciagent review" requires an intelligence backend.`);
|
||||
if (backendError) console.error(` ${backendError}`);
|
||||
process.exit(1);
|
||||
}
|
||||
@@ -394,7 +396,7 @@ export function createReviewCommand(): Command {
|
||||
phase: phaseNum,
|
||||
stage: "review",
|
||||
specification: "",
|
||||
config_path: path.join(projectPath, ".ci", "config.json"),
|
||||
config_path: path.join(projectPath, ".ciagent", "config.json"),
|
||||
backend,
|
||||
};
|
||||
|
||||
@@ -415,16 +417,16 @@ export function createStatusCommand(): Command {
|
||||
.action(() => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.log("CI project not initialized in this directory.");
|
||||
console.log("Run 'ci init' to get started.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.log("CIAgent project not initialized in this directory.");
|
||||
console.log("Run 'ciagent init' to get started.");
|
||||
return;
|
||||
}
|
||||
|
||||
const config = loadConfig(projectPath);
|
||||
const artifacts = new ArtifactManager(projectPath);
|
||||
|
||||
console.log("─── CI Project Status ───");
|
||||
console.log("─── CIAgent Project Status ───");
|
||||
console.log(`\nAutonomy: ${config.autonomy.level}`);
|
||||
console.log(`Model Profile: ${config.model_profile}`);
|
||||
console.log(`Backend: ${config.backend?.provider || "auto"}`);
|
||||
@@ -444,7 +446,7 @@ export function createStatusCommand(): Command {
|
||||
console.log(` ${icon} ${stage}`);
|
||||
}
|
||||
} else {
|
||||
console.log("\nNo pipeline state found. Run 'ci run --all' to start.");
|
||||
console.log("\nNo pipeline state found. Run 'ciagent run --all' to start.");
|
||||
}
|
||||
|
||||
const summary = getAuditSummary(projectPath);
|
||||
@@ -464,15 +466,15 @@ export function createAuditCommand(): Command {
|
||||
.action((options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
const phase = options.phase ? parseInt(options.phase) : undefined;
|
||||
const summary = getAuditSummary(projectPath);
|
||||
|
||||
console.log("─── CI Audit Report ───");
|
||||
console.log("─── CIAgent Audit Report ───");
|
||||
console.log(`\nTotal Decisions: ${summary.total_decisions}`);
|
||||
console.log(`Total Escalations: ${summary.total_escalations}`);
|
||||
console.log(`Phases Audited: ${summary.phases.join(", ") || "none"}`);
|
||||
@@ -517,8 +519,8 @@ export function createClarifyCommand(): Command {
|
||||
.action(async (options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -526,7 +528,7 @@ export function createClarifyCommand(): Command {
|
||||
const spec = loadSpec(projectPath);
|
||||
|
||||
if (!spec) {
|
||||
console.error("No specification found. Run 'ci init' first.");
|
||||
console.error("No specification found. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -557,8 +559,8 @@ export function createRollbackCommand(): Command {
|
||||
.action(async (target, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -642,6 +644,83 @@ export function createRollbackCommand(): Command {
|
||||
});
|
||||
}
|
||||
|
||||
export function createProjectsCommand(): Command {
|
||||
const cmd = new Command("projects");
|
||||
cmd.description("Manage CIAgent projects in multi-project mode");
|
||||
|
||||
cmd.command("list")
|
||||
.description("List all registered projects")
|
||||
.action(() => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
const config = loadConfig(projectPath);
|
||||
const ciFiles = new CIAgentFiles(projectPath);
|
||||
const projects = ciFiles.listProjects();
|
||||
const activeProject = config.active_project || ciFiles.getActiveProject();
|
||||
|
||||
if (projects.length === 0) {
|
||||
console.log("No projects registered.");
|
||||
console.log("Use 'ciagent projects add <slug> <name>' to add a project.");
|
||||
return;
|
||||
}
|
||||
|
||||
console.log("─── CIAgent Projects ───\n");
|
||||
for (const project of projects) {
|
||||
const isActive = project.slug === activeProject;
|
||||
const marker = isActive ? " *" : "";
|
||||
console.log(` ${project.slug} — ${project.name}${marker}`);
|
||||
}
|
||||
console.log("\n * = active project");
|
||||
});
|
||||
|
||||
cmd.command("add <slug> <name>")
|
||||
.description("Add a new project")
|
||||
.action((slug: string, name: string) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
const ciFiles = new CIAgentFiles(projectPath);
|
||||
ciFiles.addProject(slug, name);
|
||||
console.log(`✓ Project added: ${slug} (${name})`);
|
||||
});
|
||||
|
||||
cmd.command("set <slug>")
|
||||
.description("Set the active project")
|
||||
.action((slug: string) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
const ciFiles = new CIAgentFiles(projectPath);
|
||||
const projects = ciFiles.listProjects();
|
||||
|
||||
if (!projects.some((p) => p.slug === slug)) {
|
||||
console.error(`Project "${slug}" not found. Registered projects: ${projects.map((p) => p.slug).join(", ")}`);
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
ciFiles.setActiveProject(slug);
|
||||
const config = loadConfig(projectPath);
|
||||
config.active_project = slug;
|
||||
saveConfig(projectPath, config);
|
||||
console.log(`✓ Active project set to: ${slug}`);
|
||||
});
|
||||
|
||||
return cmd;
|
||||
}
|
||||
|
||||
export function createShipCommand(): Command {
|
||||
return new Command("ship")
|
||||
.description("Auto-complete phase: verify, security, commit, tag")
|
||||
@@ -650,8 +729,8 @@ export function createShipCommand(): Command {
|
||||
.action(async (phase, options) => {
|
||||
const projectPath = process.cwd();
|
||||
|
||||
if (!isCIInitialized(projectPath)) {
|
||||
console.error("CI project not initialized. Run 'ci init' first.");
|
||||
if (!isCIAgentInitialized(projectPath)) {
|
||||
console.error("CIAgent project not initialized. Run 'ciagent init' first.");
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
@@ -707,12 +786,41 @@ export function createShipCommand(): Command {
|
||||
cwd: projectPath,
|
||||
stdio: "pipe",
|
||||
});
|
||||
execSync(`git tag -a ${version.tag} -m "CI: Phase ${phaseNum} shipped"`, {
|
||||
execSync(`git tag -a ${version.tag} -m "CIAgent: Phase ${phaseNum} shipped"`, {
|
||||
cwd: projectPath,
|
||||
stdio: "pipe",
|
||||
});
|
||||
console.log(` ✓ Tagged: ${version.tag}`);
|
||||
|
||||
if (config.gitea && config.gitea.owner && config.gitea.repo) {
|
||||
const apiToken = process.env[config.gitea.api_token_env];
|
||||
if (apiToken) {
|
||||
try {
|
||||
const previousTag = getPreviousTag(projectPath, version.tag);
|
||||
const releaseNotes = generateReleaseNotes(projectPath, previousTag, version.tag);
|
||||
|
||||
const giteaClient = new GiteaClient({
|
||||
baseUrl: config.gitea.base_url,
|
||||
token: apiToken,
|
||||
owner: config.gitea.owner,
|
||||
repo: config.gitea.repo,
|
||||
});
|
||||
|
||||
const release = await giteaClient.createRelease({
|
||||
tag_name: version.tag,
|
||||
name: version.tag,
|
||||
body: releaseNotes,
|
||||
draft: false,
|
||||
prerelease: false,
|
||||
});
|
||||
|
||||
console.log(` ✓ Release created: ${release.html_url}`);
|
||||
} catch (giteaErr) {
|
||||
console.warn(` ⚠ Gitea release failed: ${giteaErr instanceof Error ? giteaErr.message : String(giteaErr)}`);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (config.git.auto_push) {
|
||||
execSync(`git push origin ${version.tag}`, { cwd: projectPath, stdio: "pipe" });
|
||||
console.log(` ✓ Pushed tag: ${version.tag}`);
|
||||
@@ -730,7 +838,7 @@ export function createShipCommand(): Command {
|
||||
function computeShipVersion(
|
||||
projectPath: string,
|
||||
phaseNum: number,
|
||||
config: CIConfig
|
||||
config: CIAgentConfig
|
||||
): { tag: string; milestoneType: "nfr" | "feature" | "schema-breaking" } {
|
||||
const tags = execSync("git tag -l", { cwd: projectPath, encoding: "utf-8" })
|
||||
.split("\n")
|
||||
@@ -819,4 +927,20 @@ function resolveMergeTarget(projectPath: string, milestoneType: string): string
|
||||
} catch {}
|
||||
|
||||
return "main";
|
||||
}
|
||||
|
||||
function getPreviousTag(projectPath: string, currentTag: string): string | null {
|
||||
try {
|
||||
const tags = execSync("git tag -l --sort=-v:refname", { cwd: projectPath, encoding: "utf-8" })
|
||||
.split("\n")
|
||||
.map((t) => t.trim())
|
||||
.filter(Boolean);
|
||||
|
||||
const currentIdx = tags.indexOf(currentTag);
|
||||
if (currentIdx >= 0 && currentIdx + 1 < tags.length) {
|
||||
return tags[currentIdx + 1];
|
||||
}
|
||||
} catch {}
|
||||
|
||||
return null;
|
||||
}
|
||||
+15
-3
@@ -2,6 +2,8 @@
|
||||
|
||||
import { Command } from "commander";
|
||||
import { VERSION } from "../version.js";
|
||||
import { CIAgentFiles } from "../core/ciagent-files.js";
|
||||
import { isCIAgentInitialized } from "../core/config.js";
|
||||
import {
|
||||
createInitCommand,
|
||||
createRunCommand,
|
||||
@@ -14,14 +16,23 @@ import {
|
||||
createClarifyCommand,
|
||||
createRollbackCommand,
|
||||
createShipCommand,
|
||||
createProjectsCommand,
|
||||
} from "./commands.js";
|
||||
|
||||
const program = new Command();
|
||||
|
||||
program
|
||||
.name("ci")
|
||||
.description("CI — Continuous Intelligence: autonomous AI-driven software engineering harness")
|
||||
.name("ciagent")
|
||||
.description("CIAgent — Continuous Intelligence: autonomous AI-driven software engineering harness")
|
||||
.version(VERSION)
|
||||
.option("--project <slug>", "Specify which project to operate on")
|
||||
.hook("preAction", () => {
|
||||
const opts = program.opts();
|
||||
if (opts.project && isCIAgentInitialized(process.cwd())) {
|
||||
const ciFiles = new CIAgentFiles(process.cwd());
|
||||
ciFiles.setProjectSlug(opts.project);
|
||||
}
|
||||
})
|
||||
.addCommand(createInitCommand())
|
||||
.addCommand(createRunCommand())
|
||||
.addCommand(createQuickCommand())
|
||||
@@ -32,6 +43,7 @@ program
|
||||
.addCommand(createAuditCommand())
|
||||
.addCommand(createClarifyCommand())
|
||||
.addCommand(createRollbackCommand())
|
||||
.addCommand(createShipCommand());
|
||||
.addCommand(createShipCommand())
|
||||
.addCommand(createProjectsCommand());
|
||||
|
||||
program.parse();
|
||||
@@ -8,7 +8,7 @@ describe("ArtifactManager", () => {
|
||||
let manager: ArtifactManager;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-artifact-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-artifact-test-"));
|
||||
manager = new ArtifactManager(tempDir);
|
||||
});
|
||||
|
||||
@@ -17,16 +17,16 @@ describe("ArtifactManager", () => {
|
||||
});
|
||||
|
||||
describe("ensureStructure", () => {
|
||||
it("creates .ci directory structure", () => {
|
||||
it("creates .ciagent directory structure", () => {
|
||||
manager.ensureStructure();
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci", "audit"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent", "audit"))).toBe(true);
|
||||
});
|
||||
|
||||
it("is idempotent", () => {
|
||||
manager.ensureStructure();
|
||||
manager.ensureStructure();
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent"))).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -67,7 +67,7 @@ describe("ArtifactManager", () => {
|
||||
|
||||
manager.writeProject(manifest);
|
||||
|
||||
const projectPath = path.join(tempDir, ".ci", "PROJECT.md");
|
||||
const projectPath = path.join(tempDir, ".ciagent", "PROJECT.md");
|
||||
expect(fs.existsSync(projectPath)).toBe(true);
|
||||
const content = fs.readFileSync(projectPath, "utf-8");
|
||||
expect(content).toContain("Test Project");
|
||||
@@ -131,7 +131,7 @@ describe("ArtifactManager", () => {
|
||||
],
|
||||
});
|
||||
|
||||
const decisionsPath = path.join(tempDir, ".ci", "DECISIONS.md");
|
||||
const decisionsPath = path.join(tempDir, ".ciagent", "DECISIONS.md");
|
||||
expect(fs.existsSync(decisionsPath)).toBe(true);
|
||||
const content = fs.readFileSync(decisionsPath, "utf-8");
|
||||
expect(content).toContain("D-001");
|
||||
|
||||
@@ -2,7 +2,7 @@ import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { writeFile, readFile, ensureDir } from "../utils/file.js";
|
||||
|
||||
const CI_DIR = ".ci";
|
||||
const CI_DIR = ".ciagent";
|
||||
|
||||
export interface ProjectManifest {
|
||||
name: string;
|
||||
|
||||
@@ -9,8 +9,8 @@ describe("Audit", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-audit-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ci", "audit"), { recursive: true });
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-audit-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ciagent", "audit"), { recursive: true });
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
@@ -40,7 +40,7 @@ describe("Audit", () => {
|
||||
],
|
||||
default_option_id: "A",
|
||||
resolution: "pending",
|
||||
audit_file: ".ci/audit/test.json",
|
||||
audit_file: ".ciagent/audit/test.json",
|
||||
};
|
||||
|
||||
describe("logDecision", () => {
|
||||
|
||||
+1
-1
@@ -12,7 +12,7 @@ export interface AuditEntry {
|
||||
const AUDIT_DIR = "audit";
|
||||
|
||||
function getAuditDir(projectPath: string): string {
|
||||
return path.join(projectPath, ".ci", AUDIT_DIR);
|
||||
return path.join(projectPath, ".ciagent", AUDIT_DIR);
|
||||
}
|
||||
|
||||
function getAuditFilePath(projectPath: string, phase: number): string {
|
||||
|
||||
@@ -1,17 +1,17 @@
|
||||
import * as os from "node:os";
|
||||
import * as path from "node:path";
|
||||
import * as fs from "node:fs";
|
||||
import { CiFiles, ProjectMd, RoadmapMd, RequirementsMd, ArchitectureMd } from "../core/ci-files.js";
|
||||
import { CIAgentFiles, ProjectMd, RoadmapMd, RequirementsMd, ArchitectureMd } from "../core/ciagent-files.js";
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ci-files-test-"));
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-files-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
describe("CiFiles", () => {
|
||||
describe("CIAgentFiles", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
@@ -22,41 +22,41 @@ describe("CiFiles", () => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
describe("ensureCIDir", () => {
|
||||
it("creates .ci directory", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
describe("ensureCIAgentDir", () => {
|
||||
it("creates .ciagent directory", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
expect(fs.existsSync(path.join(dir, ".ci"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent"))).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
describe("isInitialized", () => {
|
||||
it("returns false when no config.json exists", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
expect(ciFiles.isInitialized()).toBe(false);
|
||||
});
|
||||
|
||||
it("returns true when config.json exists", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), "{}");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), "{}");
|
||||
expect(ciFiles.isInitialized()).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
describe("projectSlug", () => {
|
||||
it("defaults to empty string", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
expect(ciFiles.getProjectSlug()).toBe("");
|
||||
});
|
||||
|
||||
it("uses provided project slug", () => {
|
||||
const ciFiles = new CiFiles(dir, "task-api");
|
||||
const ciFiles = new CIAgentFiles(dir, "task-api");
|
||||
expect(ciFiles.getProjectSlug()).toBe("task-api");
|
||||
});
|
||||
|
||||
it("setProjectSlug updates slug", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.setProjectSlug("auth-svc");
|
||||
expect(ciFiles.getProjectSlug()).toBe("auth-svc");
|
||||
});
|
||||
@@ -64,14 +64,14 @@ describe("CiFiles", () => {
|
||||
|
||||
describe("multi-project support", () => {
|
||||
it("isMultiProject returns false when not initialized", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
expect(ciFiles.isMultiProject()).toBe(false);
|
||||
});
|
||||
|
||||
it("isMultiProject returns false for single-project config", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "default", name: "Default" }],
|
||||
active_project: "default",
|
||||
}));
|
||||
@@ -79,59 +79,59 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("isMultiProject returns false for config without projects array", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({}));
|
||||
expect(ciFiles.isMultiProject()).toBe(false);
|
||||
});
|
||||
|
||||
it("addProject adds a project to config", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [],
|
||||
active_project: "",
|
||||
}));
|
||||
|
||||
ciFiles.addProject("task-api", "Task API", true);
|
||||
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ci", "config.json"), "utf-8"));
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ciagent", "config.json"), "utf-8"));
|
||||
expect(config.projects).toHaveLength(1);
|
||||
expect(config.projects[0].slug).toBe("task-api");
|
||||
expect(config.active_project).toBe("task-api");
|
||||
});
|
||||
|
||||
it("addProject does not duplicate existing project", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "task-api", name: "Task API" }],
|
||||
active_project: "task-api",
|
||||
}));
|
||||
|
||||
ciFiles.addProject("task-api", "Task API V2");
|
||||
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ci", "config.json"), "utf-8"));
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ciagent", "config.json"), "utf-8"));
|
||||
expect(config.projects).toHaveLength(1);
|
||||
});
|
||||
|
||||
it("addProject creates project subdirectory", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [],
|
||||
active_project: "",
|
||||
}));
|
||||
|
||||
ciFiles.addProject("task-api", "Task API", true);
|
||||
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "task-api"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "task-api"))).toBe(true);
|
||||
});
|
||||
|
||||
it("getActiveProject returns from config", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "task-api", name: "Task API", default: true }],
|
||||
active_project: "task-api",
|
||||
}));
|
||||
@@ -140,9 +140,9 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("setActiveProject updates config", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [
|
||||
{ slug: "task-api", name: "Task API" },
|
||||
{ slug: "auth-svc", name: "Auth Service" },
|
||||
@@ -152,14 +152,14 @@ describe("CiFiles", () => {
|
||||
|
||||
ciFiles.setActiveProject("auth-svc");
|
||||
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ci", "config.json"), "utf-8"));
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ciagent", "config.json"), "utf-8"));
|
||||
expect(config.active_project).toBe("auth-svc");
|
||||
});
|
||||
|
||||
it("listProjects returns projects from config", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [
|
||||
{ slug: "task-api", name: "Task API", default: true },
|
||||
{ slug: "auth-svc", name: "Auth Service" },
|
||||
@@ -176,71 +176,71 @@ describe("CiFiles", () => {
|
||||
|
||||
describe("needsMigration", () => {
|
||||
it("returns false when not initialized", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
expect(ciFiles.needsMigration()).toBe(false);
|
||||
});
|
||||
|
||||
it("returns false when already multi-project", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "default", name: "Default" }],
|
||||
}));
|
||||
fs.writeFileSync(path.join(dir, ".ci", "PROJECT.md"), "# Test");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "PROJECT.md"), "# Test");
|
||||
expect(ciFiles.needsMigration()).toBe(false);
|
||||
});
|
||||
|
||||
it("returns true when flat files exist without subdirs or multi-project config", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ci", "PROJECT.md"), "# Test");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "PROJECT.md"), "# Test");
|
||||
expect(ciFiles.needsMigration()).toBe(true);
|
||||
});
|
||||
|
||||
it("returns false when flat files exist but subdirs also exist", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ci", "PROJECT.md"), "# Test");
|
||||
fs.mkdirSync(path.join(dir, ".ci", "task-api"));
|
||||
fs.writeFileSync(path.join(dir, ".ci", "task-api", "PROJECT.md"), "# Task API");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "PROJECT.md"), "# Test");
|
||||
fs.mkdirSync(path.join(dir, ".ciagent", "task-api"));
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "task-api", "PROJECT.md"), "# Task API");
|
||||
expect(ciFiles.needsMigration()).toBe(false);
|
||||
});
|
||||
});
|
||||
|
||||
describe("migrateFlatToProject", () => {
|
||||
it("moves flat files to project subdirectory", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ci", "PROJECT.md"), "# Test Project");
|
||||
fs.writeFileSync(path.join(dir, ".ci", "ARCHITECTURE.md"), "# Architecture");
|
||||
fs.writeFileSync(path.join(dir, ".ci", "ROADMAP.md"), "# Roadmap");
|
||||
fs.writeFileSync(path.join(dir, ".ci", "REQUIREMENTS.md"), "# Requirements");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "PROJECT.md"), "# Test Project");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "ARCHITECTURE.md"), "# Architecture");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "ROADMAP.md"), "# Roadmap");
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "REQUIREMENTS.md"), "# Requirements");
|
||||
|
||||
ciFiles.migrateFlatToProject("my-app");
|
||||
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "my-app", "PROJECT.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "my-app", "ARCHITECTURE.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "my-app", "ROADMAP.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "my-app", "REQUIREMENTS.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "my-app", "PROJECT.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "my-app", "ARCHITECTURE.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "my-app", "ROADMAP.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "my-app", "REQUIREMENTS.md"))).toBe(true);
|
||||
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ci", "config.json"), "utf-8"));
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ciagent", "config.json"), "utf-8"));
|
||||
expect(config.projects).toHaveLength(1);
|
||||
expect(config.active_project).toBe("my-app");
|
||||
});
|
||||
|
||||
it("does not migrate when not needed", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "existing", name: "Existing" }],
|
||||
}));
|
||||
|
||||
ciFiles.migrateFlatToProject("new-proj");
|
||||
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ci", "config.json"), "utf-8"));
|
||||
const config = JSON.parse(fs.readFileSync(path.join(dir, ".ciagent", "config.json"), "utf-8"));
|
||||
expect(config.projects).toHaveLength(1);
|
||||
expect(config.projects[0].slug).toBe("existing");
|
||||
});
|
||||
@@ -248,14 +248,14 @@ describe("CiFiles", () => {
|
||||
|
||||
describe("isNfrMilestone", () => {
|
||||
it("returns true when no roadmap exists", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
expect(ciFiles.isNfrMilestone()).toBe(true);
|
||||
});
|
||||
|
||||
it("returns true when phases are all NFR types", () => {
|
||||
const ciFiles = new CiFiles(dir, "nfr-proj");
|
||||
const ciFiles = new CIAgentFiles(dir, "nfr-proj");
|
||||
ciFiles.ensureProjectDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "nfr-proj", name: "NFR Project", default: true }],
|
||||
active_project: "nfr-proj",
|
||||
}));
|
||||
@@ -271,9 +271,9 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("returns false when phases include feature work", () => {
|
||||
const ciFiles = new CiFiles(dir, "feat-proj");
|
||||
const ciFiles = new CIAgentFiles(dir, "feat-proj");
|
||||
ciFiles.ensureProjectDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "feat-proj", name: "Feature Project", default: true }],
|
||||
active_project: "feat-proj",
|
||||
}));
|
||||
@@ -291,14 +291,14 @@ describe("CiFiles", () => {
|
||||
|
||||
describe("getMilestoneType", () => {
|
||||
it("returns nfr when no roadmap exists", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
expect(ciFiles.getMilestoneType()).toBe("nfr");
|
||||
});
|
||||
|
||||
it("returns nfr when phases are all NFR types", () => {
|
||||
const ciFiles = new CiFiles(dir, "nfr-proj2");
|
||||
const ciFiles = new CIAgentFiles(dir, "nfr-proj2");
|
||||
ciFiles.ensureProjectDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "nfr-proj2", name: "NFR Project 2", default: true }],
|
||||
active_project: "nfr-proj2",
|
||||
}));
|
||||
@@ -313,9 +313,9 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("returns feature when phases include feat work", () => {
|
||||
const ciFiles = new CiFiles(dir, "feat-proj2");
|
||||
const ciFiles = new CIAgentFiles(dir, "feat-proj2");
|
||||
ciFiles.ensureProjectDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "feat-proj2", name: "Feature Project 2", default: true }],
|
||||
active_project: "feat-proj2",
|
||||
}));
|
||||
@@ -330,9 +330,9 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("returns schema-breaking when phases include refactor/rewrite/migrate", () => {
|
||||
const ciFiles = new CiFiles(dir, "schema-proj");
|
||||
const ciFiles = new CIAgentFiles(dir, "schema-proj");
|
||||
ciFiles.ensureProjectDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "schema-proj", name: "Schema Project", default: true }],
|
||||
active_project: "schema-proj",
|
||||
}));
|
||||
@@ -349,9 +349,9 @@ describe("CiFiles", () => {
|
||||
|
||||
describe("multi-project file paths", () => {
|
||||
it("writes PROJECT.md to project subdirectory when slug is set", () => {
|
||||
const ciFiles = new CiFiles(dir, "my-app");
|
||||
const ciFiles = new CIAgentFiles(dir, "my-app");
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({
|
||||
projects: [{ slug: "my-app", name: "My App", default: true }],
|
||||
active_project: "my-app",
|
||||
}));
|
||||
@@ -367,13 +367,13 @@ describe("CiFiles", () => {
|
||||
|
||||
ciFiles.writeProjectMd(project, "initial");
|
||||
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "my-app", "PROJECT.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "my-app", "PROJECT.md"))).toBe(true);
|
||||
});
|
||||
|
||||
it("writes PROJECT.md to .ci root when no slug is set", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
fs.writeFileSync(path.join(dir, ".ci", "config.json"), JSON.stringify({}));
|
||||
fs.writeFileSync(path.join(dir, ".ciagent", "config.json"), JSON.stringify({}));
|
||||
|
||||
const project: ProjectMd = {
|
||||
name: "Default App",
|
||||
@@ -386,7 +386,7 @@ describe("CiFiles", () => {
|
||||
|
||||
ciFiles.writeProjectMd(project, "initial");
|
||||
|
||||
expect(fs.existsSync(path.join(dir, ".ci", "PROJECT.md"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "PROJECT.md"))).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -407,7 +407,7 @@ describe("CiFiles", () => {
|
||||
};
|
||||
|
||||
it("writes and reads PROJECT.md", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.writeProjectMd(project, "initial creation");
|
||||
|
||||
const read = ciFiles.readProjectMd();
|
||||
@@ -418,7 +418,7 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("overwrites PROJECT.md on update", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.writeProjectMd(project, "initial");
|
||||
|
||||
const updated = { ...project, coreValue: "Updated description" };
|
||||
@@ -455,7 +455,7 @@ describe("CiFiles", () => {
|
||||
};
|
||||
|
||||
it("writes and reads ROADMAP.md", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.writeRoadmapMd(roadmap);
|
||||
|
||||
const read = ciFiles.readRoadmapMd();
|
||||
@@ -489,7 +489,7 @@ describe("CiFiles", () => {
|
||||
};
|
||||
|
||||
it("writes and reads REQUIREMENTS.md", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.writeRequirementsMd(requirements);
|
||||
|
||||
const read = ciFiles.readRequirementsMd();
|
||||
@@ -497,7 +497,7 @@ describe("CiFiles", () => {
|
||||
});
|
||||
|
||||
it("updates requirement status", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.writeRequirementsMd(requirements);
|
||||
|
||||
ciFiles.updateRequirementStatus("AUTH-01", "complete");
|
||||
@@ -523,7 +523,7 @@ describe("CiFiles", () => {
|
||||
};
|
||||
|
||||
it("writes and reads ARCHITECTURE.md", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.writeArchitectureMd(arch);
|
||||
|
||||
const read = ciFiles.readArchitectureMd();
|
||||
@@ -534,7 +534,7 @@ describe("CiFiles", () => {
|
||||
|
||||
describe("updatePhaseStatus", () => {
|
||||
it("updates phase status in roadmap", () => {
|
||||
const ciFiles = new CiFiles(dir);
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
const roadmap: RoadmapMd = {
|
||||
overview: "test",
|
||||
phases: [
|
||||
@@ -4,7 +4,7 @@ import { writeFile, readFile, ensureDir, fileExists } from "../utils/file.js";
|
||||
import { PipelineStage } from "../types/pipeline.js";
|
||||
import { MilestoneType } from "../types/config.js";
|
||||
|
||||
const CI_DIR = ".ci";
|
||||
const CI_DIR = ".ciagent";
|
||||
|
||||
export interface ProjectMd {
|
||||
name: string;
|
||||
@@ -71,7 +71,7 @@ export interface ProjectEntry {
|
||||
default?: boolean;
|
||||
}
|
||||
|
||||
export class CiFiles {
|
||||
export class CIAgentFiles {
|
||||
private projectPath: string;
|
||||
private projectSlug: string;
|
||||
|
||||
+18
-18
@@ -2,15 +2,15 @@ import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { ClarifyPhase, saveSpecification, loadSpecification } from "../core/clarify.js";
|
||||
import { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
import { Specification, parseSpecification } from "../types/specification.js";
|
||||
|
||||
describe("ClarifyPhase", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-clarify-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ci"), { recursive: true });
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-clarify-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ciagent"), { recursive: true });
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
@@ -41,7 +41,7 @@ describe("ClarifyPhase", () => {
|
||||
|
||||
describe("generateQuestions", () => {
|
||||
it("generates questions for missing requirements", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
expect(questions.length).toBeGreaterThan(0);
|
||||
const reqQuestion = questions.find((q) => q.category === "requirements");
|
||||
@@ -50,7 +50,7 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("generates questions for missing constraints", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
const constraintQuestion = questions.find((q) => q.category === "constraints");
|
||||
expect(constraintQuestion).toBeDefined();
|
||||
@@ -58,7 +58,7 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("generates deployment question when deploy is mentioned without deploy constraint", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithRequirements);
|
||||
const deployQuestion = questions.find((q) => q.category === "deployment");
|
||||
expect(deployQuestion).toBeDefined();
|
||||
@@ -66,8 +66,8 @@ describe("ClarifyPhase", () => {
|
||||
|
||||
it("respects clarify_budget", () => {
|
||||
const limitedConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, clarify_budget: 1 },
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, clarify_budget: 1 },
|
||||
};
|
||||
const clarify = new ClarifyPhase(limitedConfig, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
@@ -75,7 +75,7 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("assigns sequential question IDs", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
for (let i = 0; i < questions.length; i++) {
|
||||
expect(questions[i].id).toBe(`Q-${String(i + 1).padStart(3, "0")}`);
|
||||
@@ -83,7 +83,7 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("sorts questions by impact priority", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
const priorityOrder = { critical: 0, high: 1, medium: 2, low: 3 };
|
||||
for (let i = 1; i < questions.length; i++) {
|
||||
@@ -96,7 +96,7 @@ describe("ClarifyPhase", () => {
|
||||
|
||||
describe("answerQuestion", () => {
|
||||
it("records an answer to a question", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
expect(questions.length).toBeGreaterThan(0);
|
||||
|
||||
@@ -107,7 +107,7 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("returns null for unknown question ID", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const result = clarify.answerQuestion("Q-999", "answer");
|
||||
expect(result).toBeNull();
|
||||
});
|
||||
@@ -115,7 +115,7 @@ describe("ClarifyPhase", () => {
|
||||
|
||||
describe("acceptDefaults", () => {
|
||||
it("accepts defaults for all unanswered questions", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
clarify.generateQuestions(specWithoutRequirements);
|
||||
const result = clarify.acceptDefaults();
|
||||
|
||||
@@ -125,7 +125,7 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("preserves manually answered questions", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
const questions = clarify.generateQuestions(specWithoutRequirements);
|
||||
if (questions.length > 0) {
|
||||
clarify.answerQuestion(questions[0].id, "My answer");
|
||||
@@ -138,11 +138,11 @@ describe("ClarifyPhase", () => {
|
||||
});
|
||||
|
||||
it("saves clarify responses file", () => {
|
||||
const clarify = new ClarifyPhase(DEFAULT_CI_CONFIG, tempDir);
|
||||
const clarify = new ClarifyPhase(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
clarify.generateQuestions(specWithoutRequirements);
|
||||
clarify.acceptDefaults();
|
||||
|
||||
const responsesPath = path.join(tempDir, ".ci", "clarify-responses.md");
|
||||
const responsesPath = path.join(tempDir, ".ciagent", "clarify-responses.md");
|
||||
expect(fs.existsSync(responsesPath)).toBe(true);
|
||||
const content = fs.readFileSync(responsesPath, "utf-8");
|
||||
expect(content).toContain("Clarify Phase Responses");
|
||||
@@ -154,8 +154,8 @@ describe("saveSpecification / loadSpecification", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-spec-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ci"), { recursive: true });
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-spec-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ciagent"), { recursive: true });
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
|
||||
+4
-4
@@ -2,22 +2,22 @@ import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { ClarifyQuestion, ClarifyResult } from "../types/clarify.js";
|
||||
import { Specification, parseSpecification } from "../types/specification.js";
|
||||
import { CIConfig } from "../types/config.js";
|
||||
import { CIAgentConfig } from "../types/config.js";
|
||||
|
||||
const CLARIFY_RESPONSES_FILE = "clarify-responses.md";
|
||||
const SPECIFICATION_FILE = "specification.md";
|
||||
|
||||
function getCIDir(projectPath: string): string {
|
||||
return path.join(projectPath, ".ci");
|
||||
return path.join(projectPath, ".ciagent");
|
||||
}
|
||||
|
||||
export class ClarifyPhase {
|
||||
private config: CIConfig;
|
||||
private config: CIAgentConfig;
|
||||
private projectPath: string;
|
||||
private questions: ClarifyQuestion[];
|
||||
private questionCounter: number;
|
||||
|
||||
constructor(config: CIConfig, projectPath: string) {
|
||||
constructor(config: CIAgentConfig, projectPath: string) {
|
||||
this.config = config;
|
||||
this.projectPath = projectPath;
|
||||
this.questions = [];
|
||||
|
||||
@@ -1,11 +1,11 @@
|
||||
import { CommitBuilder } from "../core/commit-builder.js";
|
||||
import { extractCiBlock, parseCiBlock } from "../core/commit-parser.js";
|
||||
import { CiMetadata } from "../types/commit-meta.js";
|
||||
import { extractCIAgentBlock, parseCIAgentBlock } from "../core/commit-parser.js";
|
||||
import { CIAgentMetadata } from "../types/commit-meta.js";
|
||||
|
||||
describe("CommitBuilder", () => {
|
||||
describe("buildCiBlock", () => {
|
||||
it("builds minimal ci block", () => {
|
||||
const ci: CiMetadata = { phase: 1, milestone: "v1.0", status: "execute" };
|
||||
const ci: CIAgentMetadata = { phase: 1, milestone: "v1.0", status: "execute" };
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
|
||||
expect(block).toContain("phase: 1");
|
||||
@@ -14,19 +14,19 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("builds ci block with project", () => {
|
||||
const ci: CiMetadata = { phase: 1, milestone: "v1.0", status: "execute", project: "task-api" };
|
||||
const ci: CIAgentMetadata = { phase: 1, milestone: "v1.0", status: "execute", project: "task-api" };
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
expect(block).toContain("project: task-api");
|
||||
});
|
||||
|
||||
it("builds ci block without project when not set", () => {
|
||||
const ci: CiMetadata = { phase: 1, milestone: "v1.0", status: "execute" };
|
||||
const ci: CIAgentMetadata = { phase: 1, milestone: "v1.0", status: "execute" };
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
expect(block).not.toContain("project:");
|
||||
});
|
||||
|
||||
it("builds ci block with decisions", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 1,
|
||||
milestone: "v1.0",
|
||||
status: "execute",
|
||||
@@ -49,7 +49,7 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("builds ci block with lessons", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 1,
|
||||
milestone: "v1.0",
|
||||
status: "complete",
|
||||
@@ -63,7 +63,7 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("builds ci block with compound", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 1,
|
||||
milestone: "v1.0",
|
||||
status: "complete",
|
||||
@@ -82,7 +82,7 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("builds ci block with escalations", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 3,
|
||||
milestone: "v1.0",
|
||||
status: "execute",
|
||||
@@ -103,7 +103,7 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("builds ci block with requirements", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 1,
|
||||
milestone: "v1.0",
|
||||
status: "complete",
|
||||
@@ -122,12 +122,12 @@ describe("CommitBuilder", () => {
|
||||
|
||||
describe("round-trip: build then parse", () => {
|
||||
it("round-trips a simple ci block", () => {
|
||||
const ci: CiMetadata = { phase: 1, milestone: "v1.0", status: "execute" };
|
||||
const ci: CIAgentMetadata = { phase: 1, milestone: "v1.0", status: "execute" };
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
|
||||
const fullMessage = `feat(P01): test\n\n---ci---\n${block}\n---/ci---\n\nBody text`;
|
||||
const extracted = extractCiBlock(fullMessage)!;
|
||||
const parsed = parseCiBlock(extracted)!;
|
||||
const extracted = extractCIAgentBlock(fullMessage)!;
|
||||
const parsed = parseCIAgentBlock(extracted)!;
|
||||
|
||||
expect(parsed.phase).toBe(1);
|
||||
expect(parsed.milestone).toBe("v1.0");
|
||||
@@ -135,7 +135,7 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("round-trips decisions", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 1,
|
||||
milestone: "v1.0",
|
||||
status: "execute",
|
||||
@@ -152,8 +152,8 @@ describe("CommitBuilder", () => {
|
||||
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
const fullMessage = `feat(P01): test\n\n---ci---\n${block}\n---/ci---`;
|
||||
const extracted = extractCiBlock(fullMessage)!;
|
||||
const parsed = parseCiBlock(extracted)!;
|
||||
const extracted = extractCIAgentBlock(fullMessage)!;
|
||||
const parsed = parseCIAgentBlock(extracted)!;
|
||||
|
||||
expect(parsed.decisions).toHaveLength(1);
|
||||
expect(parsed.decisions![0].id).toBe("D-001");
|
||||
@@ -163,7 +163,7 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("round-trips compound with lessons", () => {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 2,
|
||||
milestone: "v1.0",
|
||||
status: "complete",
|
||||
@@ -177,8 +177,8 @@ describe("CommitBuilder", () => {
|
||||
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
const fullMessage = `compound(P02): test\n\n---ci---\n${block}\n---/ci---`;
|
||||
const extracted = extractCiBlock(fullMessage)!;
|
||||
const parsed = parseCiBlock(extracted)!;
|
||||
const extracted = extractCIAgentBlock(fullMessage)!;
|
||||
const parsed = parseCIAgentBlock(extracted)!;
|
||||
|
||||
expect(parsed.compound!.category).toBe("auth");
|
||||
expect(parsed.compound!.problem).toBe("Token replay attacks");
|
||||
@@ -186,11 +186,11 @@ describe("CommitBuilder", () => {
|
||||
});
|
||||
|
||||
it("round-trips project field", () => {
|
||||
const ci: CiMetadata = { phase: 1, milestone: "v1.0", status: "execute", project: "task-api" };
|
||||
const ci: CIAgentMetadata = { phase: 1, milestone: "v1.0", status: "execute", project: "task-api" };
|
||||
const block = CommitBuilder.buildCiBlock(ci);
|
||||
const fullMessage = `feat(task-api/P01): test\n\n---ci---\n${block}\n---/ci---`;
|
||||
const extracted = extractCiBlock(fullMessage)!;
|
||||
const parsed = parseCiBlock(extracted)!;
|
||||
const extracted = extractCIAgentBlock(fullMessage)!;
|
||||
const parsed = parseCIAgentBlock(extracted)!;
|
||||
|
||||
expect(parsed.project).toBe("task-api");
|
||||
});
|
||||
|
||||
+11
-11
@@ -1,5 +1,5 @@
|
||||
import {
|
||||
CiMetadata,
|
||||
CIAgentMetadata,
|
||||
CommitType,
|
||||
CommitScope,
|
||||
CommitDecision,
|
||||
@@ -17,7 +17,7 @@ export interface CommitMessageInput {
|
||||
type: CommitType;
|
||||
scope: CommitScope;
|
||||
subject: string;
|
||||
ci: CiMetadata;
|
||||
ci: CIAgentMetadata;
|
||||
body?: string;
|
||||
}
|
||||
|
||||
@@ -92,7 +92,7 @@ export interface VerifyCommitInput {
|
||||
}
|
||||
|
||||
export class CommitBuilder {
|
||||
static buildCiBlock(ci: CiMetadata): string {
|
||||
static buildCiBlock(ci: CIAgentMetadata): string {
|
||||
const lines: string[] = [];
|
||||
lines.push(`phase: ${ci.phase}`);
|
||||
lines.push(`milestone: ${ci.milestone}`);
|
||||
@@ -162,7 +162,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildInitCommit(input: InitCommitInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: 0,
|
||||
milestone: input.milestone,
|
||||
project: input.project,
|
||||
@@ -194,7 +194,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildTaskCommit(input: TaskCommitInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: input.phase,
|
||||
milestone: input.milestone,
|
||||
project: input.project,
|
||||
@@ -224,7 +224,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildPhaseCompletionCommit(input: PhaseCompletionInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: input.phase,
|
||||
milestone: input.milestone,
|
||||
status: "complete",
|
||||
@@ -253,7 +253,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildDecisionCommit(input: DecisionCommitInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: input.phase,
|
||||
milestone: input.milestone,
|
||||
status: "plan",
|
||||
@@ -271,7 +271,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildEscalationCommit(input: EscalationCommitInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: input.phase,
|
||||
milestone: input.milestone,
|
||||
status: "execute",
|
||||
@@ -289,7 +289,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildCompoundCommit(input: CompoundCommitInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: input.phase,
|
||||
milestone: input.milestone,
|
||||
status: "complete",
|
||||
@@ -313,7 +313,7 @@ export class CommitBuilder {
|
||||
}
|
||||
|
||||
static buildVerifyCommit(input: VerifyCommitInput): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase: input.phase,
|
||||
milestone: input.milestone,
|
||||
status: "verify",
|
||||
@@ -338,7 +338,7 @@ export class CommitBuilder {
|
||||
findings: string[],
|
||||
decisions?: CommitDecision[]
|
||||
): string {
|
||||
const ci: CiMetadata = {
|
||||
const ci: CIAgentMetadata = {
|
||||
phase,
|
||||
milestone,
|
||||
status: "research",
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
import {
|
||||
CiMetadata,
|
||||
CIAgentMetadata,
|
||||
CommitDecision,
|
||||
CommitEscalation,
|
||||
CommitRequirements,
|
||||
@@ -9,8 +9,8 @@ import {
|
||||
CommitScope,
|
||||
} from "../types/commit-meta.js";
|
||||
import {
|
||||
extractCiBlock,
|
||||
parseCiBlock,
|
||||
extractCIAgentBlock,
|
||||
parseCIAgentBlock,
|
||||
parseCommitMessage,
|
||||
} from "./commit-parser.js";
|
||||
|
||||
@@ -128,29 +128,29 @@ status: execute
|
||||
|
||||
Registration endpoint for task-api project.`;
|
||||
|
||||
describe("extractCiBlock", () => {
|
||||
describe("extractCIAgentBlock", () => {
|
||||
it("extracts ---ci--- block from commit message", () => {
|
||||
const block = extractCiBlock(SAMPLE_INIT_COMMIT);
|
||||
const block = extractCIAgentBlock(SAMPLE_INIT_COMMIT);
|
||||
expect(block).toBeTruthy();
|
||||
expect(block).toContain("phase: 0");
|
||||
expect(block).toContain("milestone: v1.0");
|
||||
});
|
||||
|
||||
it("returns null when no ---ci--- block exists", () => {
|
||||
const block = extractCiBlock("docs: some regular commit\n\nNo CI block here");
|
||||
const block = extractCIAgentBlock("docs: some regular commit\n\nNo CI block here");
|
||||
expect(block).toBeNull();
|
||||
});
|
||||
|
||||
it("returns null for unclosed ---ci--- block", () => {
|
||||
const block = extractCiBlock("docs: bad\n---ci---\nphase: 1\nno end marker");
|
||||
const block = extractCIAgentBlock("docs: bad\n---ci---\nphase: 1\nno end marker");
|
||||
expect(block).toBeNull();
|
||||
});
|
||||
});
|
||||
|
||||
describe("parseCiBlock", () => {
|
||||
describe("parseCIAgentBlock", () => {
|
||||
it("parses init commit ci block", () => {
|
||||
const block = extractCiBlock(SAMPLE_INIT_COMMIT)!;
|
||||
const meta = parseCiBlock(block)!;
|
||||
const block = extractCIAgentBlock(SAMPLE_INIT_COMMIT)!;
|
||||
const meta = parseCIAgentBlock(block)!;
|
||||
|
||||
expect(meta.phase).toBe(0);
|
||||
expect(meta.milestone).toBe("v1.0");
|
||||
@@ -163,8 +163,8 @@ describe("parseCiBlock", () => {
|
||||
});
|
||||
|
||||
it("parses task commit ci block", () => {
|
||||
const block = extractCiBlock(SAMPLE_TASK_COMMIT)!;
|
||||
const meta = parseCiBlock(block)!;
|
||||
const block = extractCIAgentBlock(SAMPLE_TASK_COMMIT)!;
|
||||
const meta = parseCIAgentBlock(block)!;
|
||||
|
||||
expect(meta.phase).toBe(1);
|
||||
expect(meta.plan).toBe("01-01");
|
||||
@@ -177,8 +177,8 @@ describe("parseCiBlock", () => {
|
||||
});
|
||||
|
||||
it("parses phase completion with lessons", () => {
|
||||
const block = extractCiBlock(SAMPLE_PHASE_COMPLETE_COMMIT)!;
|
||||
const meta = parseCiBlock(block)!;
|
||||
const block = extractCIAgentBlock(SAMPLE_PHASE_COMPLETE_COMMIT)!;
|
||||
const meta = parseCIAgentBlock(block)!;
|
||||
|
||||
expect(meta.phase).toBe(1);
|
||||
expect(meta.status).toBe("complete");
|
||||
@@ -188,8 +188,8 @@ describe("parseCiBlock", () => {
|
||||
});
|
||||
|
||||
it("parses compound commit", () => {
|
||||
const block = extractCiBlock(SAMPLE_COMPOUND_COMMIT)!;
|
||||
const meta = parseCiBlock(block)!;
|
||||
const block = extractCIAgentBlock(SAMPLE_COMPOUND_COMMIT)!;
|
||||
const meta = parseCIAgentBlock(block)!;
|
||||
|
||||
expect(meta.compound).toBeDefined();
|
||||
expect(meta.compound!.category).toBe("auth");
|
||||
@@ -199,8 +199,8 @@ describe("parseCiBlock", () => {
|
||||
});
|
||||
|
||||
it("parses escalation commit", () => {
|
||||
const block = extractCiBlock(SAMPLE_ESCALATION_COMMIT)!;
|
||||
const meta = parseCiBlock(block)!;
|
||||
const block = extractCIAgentBlock(SAMPLE_ESCALATION_COMMIT)!;
|
||||
const meta = parseCIAgentBlock(block)!;
|
||||
|
||||
expect(meta.escalations).toHaveLength(1);
|
||||
expect(meta.escalations![0].id).toBe("E-001");
|
||||
@@ -209,20 +209,20 @@ describe("parseCiBlock", () => {
|
||||
});
|
||||
|
||||
it("parses project field", () => {
|
||||
const block = extractCiBlock(SAMPLE_PROJECT_COMMIT)!;
|
||||
const meta = parseCiBlock(block)!;
|
||||
const block = extractCIAgentBlock(SAMPLE_PROJECT_COMMIT)!;
|
||||
const meta = parseCIAgentBlock(block)!;
|
||||
expect(meta.project).toBe("task-api");
|
||||
expect(meta.phase).toBe(1);
|
||||
expect(meta.plan).toBe("01-01");
|
||||
});
|
||||
|
||||
it("returns null for empty block", () => {
|
||||
const meta = parseCiBlock("");
|
||||
const meta = parseCIAgentBlock("");
|
||||
expect(meta).toBeNull();
|
||||
});
|
||||
|
||||
it("returns null for block missing required fields", () => {
|
||||
const meta = parseCiBlock("something: true\nother: false");
|
||||
const meta = parseCIAgentBlock("something: true\nother: false");
|
||||
expect(meta).toBeNull();
|
||||
});
|
||||
});
|
||||
|
||||
+17
-17
@@ -1,8 +1,8 @@
|
||||
import {
|
||||
CiMetadata,
|
||||
CIAgentMetadata,
|
||||
CommitType,
|
||||
CommitEscalation,
|
||||
ParsedCiCommit,
|
||||
ParsedCIAgentCommit,
|
||||
parseCommitType,
|
||||
parseCommitScope,
|
||||
} from "../types/commit-meta.js";
|
||||
@@ -10,7 +10,7 @@ import {
|
||||
const CI_BLOCK_START = "---ci---";
|
||||
const CI_BLOCK_END = "---/ci---";
|
||||
|
||||
export function extractCiBlock(message: string): string | null {
|
||||
export function extractCIAgentBlock(message: string): string | null {
|
||||
const startIdx = message.indexOf(CI_BLOCK_START);
|
||||
if (startIdx < 0) return null;
|
||||
|
||||
@@ -20,10 +20,10 @@ export function extractCiBlock(message: string): string | null {
|
||||
return message.slice(startIdx + CI_BLOCK_START.length, endIdx).trim();
|
||||
}
|
||||
|
||||
export function parseCiBlock(yaml: string): CiMetadata | null {
|
||||
export function parseCIAgentBlock(yaml: string): CIAgentMetadata | null {
|
||||
if (!yaml) return null;
|
||||
|
||||
const result: Partial<CiMetadata> = {};
|
||||
const result: Partial<CIAgentMetadata> = {};
|
||||
|
||||
const phaseMatch = yaml.match(/^phase:\s*(.+)$/m);
|
||||
if (phaseMatch) result.phase = parseInt(phaseMatch[1], 10) || 0;
|
||||
@@ -38,7 +38,7 @@ export function parseCiBlock(yaml: string): CiMetadata | null {
|
||||
if (taskMatch) result.task = taskMatch[1].trim();
|
||||
|
||||
const statusMatch = yaml.match(/^status:\s*(.+)$/m);
|
||||
if (statusMatch) result.status = statusMatch[1].trim() as CiMetadata["status"];
|
||||
if (statusMatch) result.status = statusMatch[1].trim() as CIAgentMetadata["status"];
|
||||
|
||||
const projectMatch = yaml.match(/^project:\s*(.+)$/m);
|
||||
if (projectMatch) result.project = projectMatch[1].trim();
|
||||
@@ -50,14 +50,14 @@ export function parseCiBlock(yaml: string): CiMetadata | null {
|
||||
result.compound = parseCompoundFromYaml(yaml);
|
||||
|
||||
if (result.phase !== undefined && result.milestone !== undefined && result.status !== undefined) {
|
||||
return result as CiMetadata;
|
||||
return result as CIAgentMetadata;
|
||||
}
|
||||
|
||||
return null;
|
||||
}
|
||||
|
||||
function parseDecisionsFromYaml(yaml: string): CiMetadata["decisions"] {
|
||||
const decisions: NonNullable<CiMetadata["decisions"]> = [];
|
||||
function parseDecisionsFromYaml(yaml: string): CIAgentMetadata["decisions"] {
|
||||
const decisions: NonNullable<CIAgentMetadata["decisions"]> = [];
|
||||
const decisionRegex = /- id: (.+)\n\s+decision: (.+)\n\s+rationale: (.+)\n\s+confidence: (.+)\n\s+alternatives: \[([^\]]*)\]/g;
|
||||
let match;
|
||||
|
||||
@@ -74,8 +74,8 @@ function parseDecisionsFromYaml(yaml: string): CiMetadata["decisions"] {
|
||||
return decisions.length > 0 ? decisions : undefined;
|
||||
}
|
||||
|
||||
function parseEscalationsFromYaml(yaml: string): CiMetadata["escalations"] {
|
||||
const escalations: NonNullable<CiMetadata["escalations"]> = [];
|
||||
function parseEscalationsFromYaml(yaml: string): CIAgentMetadata["escalations"] {
|
||||
const escalations: NonNullable<CIAgentMetadata["escalations"]> = [];
|
||||
const escalationRegex = /- id: (.+)\n\s+type: (.+)\n\s+description: (.+)\n\s+resolution: (.+)/g;
|
||||
let match;
|
||||
|
||||
@@ -91,7 +91,7 @@ function parseEscalationsFromYaml(yaml: string): CiMetadata["escalations"] {
|
||||
return escalations.length > 0 ? escalations : undefined;
|
||||
}
|
||||
|
||||
function parseRequirementsFromYaml(yaml: string): CiMetadata["requirements"] {
|
||||
function parseRequirementsFromYaml(yaml: string): CIAgentMetadata["requirements"] {
|
||||
const coveredMatch = yaml.match(/^\s+covered: \[([^\]]*)\]/m);
|
||||
const partialMatch = yaml.match(/^\s+partial: \[([^\]]*)\]/m);
|
||||
|
||||
@@ -106,7 +106,7 @@ function parseRequirementsFromYaml(yaml: string): CiMetadata["requirements"] {
|
||||
return { covered, partial };
|
||||
}
|
||||
|
||||
function parseLessonsFromYaml(yaml: string): CiMetadata["lessons"] {
|
||||
function parseLessonsFromYaml(yaml: string): CIAgentMetadata["lessons"] {
|
||||
const lessonRegex = /^ - (.+)$/gm;
|
||||
const lessons: string[] = [];
|
||||
let inLessonsSection = false;
|
||||
@@ -126,7 +126,7 @@ function parseLessonsFromYaml(yaml: string): CiMetadata["lessons"] {
|
||||
return lessons.length > 0 ? lessons : undefined;
|
||||
}
|
||||
|
||||
function parseCompoundFromYaml(yaml: string): CiMetadata["compound"] {
|
||||
function parseCompoundFromYaml(yaml: string): CIAgentMetadata["compound"] {
|
||||
const categoryMatch = yaml.match(/^\s+category: (.+)$/m);
|
||||
const problemMatch = yaml.match(/^\s+problem: (.+)$/m);
|
||||
const solutionMatch = yaml.match(/^\s+solution: (.+)$/m);
|
||||
@@ -143,7 +143,7 @@ function parseCompoundFromYaml(yaml: string): CiMetadata["compound"] {
|
||||
export function parseCommitMessage(
|
||||
hash: string,
|
||||
message: string
|
||||
): ParsedCiCommit {
|
||||
): ParsedCIAgentCommit {
|
||||
const firstLine = message.split("\n")[0] || "";
|
||||
const subjectMatch = firstLine.match(/^(\w+)(?:\(([^)]+)\))?: (.+)$/);
|
||||
|
||||
@@ -157,8 +157,8 @@ export function parseCommitMessage(
|
||||
subject = subjectMatch[3] || firstLine;
|
||||
}
|
||||
|
||||
const ciBlock = extractCiBlock(message);
|
||||
const ci = ciBlock ? parseCiBlock(ciBlock) : null;
|
||||
const ciBlock = extractCIAgentBlock(message);
|
||||
const ci = ciBlock ? parseCIAgentBlock(ciBlock) : null;
|
||||
|
||||
const bodyStart = message.indexOf("\n");
|
||||
let body = bodyStart >= 0 ? message.slice(bodyStart + 1).trim() : "";
|
||||
|
||||
+37
-37
@@ -1,45 +1,45 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { initCI, loadConfig, saveConfig, isCIInitialized, ensureCIDir } from "../core/config.js";
|
||||
import { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { initCIAgent, loadConfig, saveConfig, isCIAgentInitialized, ensureCIDir } from "../core/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
describe("CI Config", () => {
|
||||
describe("CIAgent Config", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-config-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-config-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
fs.rmSync(tempDir, { recursive: true, force: true });
|
||||
});
|
||||
|
||||
describe("initCI", () => {
|
||||
it("initializes a new CI project with default config", () => {
|
||||
const config = initCI(tempDir);
|
||||
describe("initCIAgent", () => {
|
||||
it("initializes a new CIAgent project with default config", () => {
|
||||
const config = initCIAgent(tempDir);
|
||||
expect(config.autonomy.level).toBe("full");
|
||||
expect(isCIInitialized(tempDir)).toBe(true);
|
||||
expect(isCIAgentInitialized(tempDir)).toBe(true);
|
||||
});
|
||||
|
||||
it("initializes with custom config merged on top of defaults", () => {
|
||||
const config = initCI(tempDir, {
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, level: "guided" },
|
||||
const config = initCIAgent(tempDir, {
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, level: "guided" },
|
||||
});
|
||||
expect(config.autonomy.level).toBe("guided");
|
||||
expect(config.autonomy.clarify_budget).toBe(10);
|
||||
expect(config.model_profile).toBe("quality");
|
||||
});
|
||||
|
||||
it("creates .ci/ directory structure", () => {
|
||||
initCI(tempDir);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci", "config.json"))).toBe(true);
|
||||
it("creates .ciagent/ directory structure", () => {
|
||||
initCIAgent(tempDir);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent", "config.json"))).toBe(true);
|
||||
});
|
||||
|
||||
it("deep merges nested config", () => {
|
||||
const config = initCI(tempDir, {
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, level: "supervised" },
|
||||
const config = initCIAgent(tempDir, {
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, level: "supervised" },
|
||||
});
|
||||
expect(config.autonomy.level).toBe("supervised");
|
||||
expect(config.autonomy.max_revision_iterations).toBe(3);
|
||||
@@ -47,7 +47,7 @@ describe("CI Config", () => {
|
||||
});
|
||||
|
||||
it("initializes with project slug", () => {
|
||||
const config = initCI(tempDir, undefined, "task-api", "Task API");
|
||||
const config = initCIAgent(tempDir, undefined, "task-api", "Task API");
|
||||
expect(config.projects).toHaveLength(1);
|
||||
expect(config.projects[0].slug).toBe("task-api");
|
||||
expect(config.projects[0].name).toBe("Task API");
|
||||
@@ -56,20 +56,20 @@ describe("CI Config", () => {
|
||||
});
|
||||
|
||||
it("does not re-add existing project slug", () => {
|
||||
initCI(tempDir, undefined, "task-api", "Task API");
|
||||
const config = initCI(tempDir, undefined, "task-api", "Task API V2");
|
||||
initCIAgent(tempDir, undefined, "task-api", "Task API");
|
||||
const config = initCIAgent(tempDir, undefined, "task-api", "Task API V2");
|
||||
expect(config.projects).toHaveLength(1);
|
||||
});
|
||||
|
||||
it("defaults projects and active_project when no slug provided", () => {
|
||||
const config = initCI(tempDir);
|
||||
const config = initCIAgent(tempDir);
|
||||
expect(config.projects).toEqual([]);
|
||||
expect(config.active_project).toBe("");
|
||||
});
|
||||
|
||||
it("preserves existing projects when adding new one", () => {
|
||||
const config1 = initCI(tempDir, undefined, "task-api", "Task API");
|
||||
const config2 = initCI(tempDir, {
|
||||
const config1 = initCIAgent(tempDir, undefined, "task-api", "Task API");
|
||||
const config2 = initCIAgent(tempDir, {
|
||||
...config1,
|
||||
projects: [...config1.projects, { slug: "auth-svc", name: "Auth Service" }],
|
||||
}, "auth-svc", "Auth Service");
|
||||
@@ -81,11 +81,11 @@ describe("CI Config", () => {
|
||||
describe("loadConfig", () => {
|
||||
it("returns default config when no config file exists", () => {
|
||||
const config = loadConfig(tempDir);
|
||||
expect(config).toEqual(DEFAULT_CI_CONFIG);
|
||||
expect(config).toEqual(DEFAULT_CIAGENT_CONFIG);
|
||||
});
|
||||
|
||||
it("loads and deep merges config from file", () => {
|
||||
initCI(tempDir, { autonomy: { ...DEFAULT_CI_CONFIG.autonomy, decision_confidence_threshold: 0.85 } });
|
||||
initCIAgent(tempDir, { autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, decision_confidence_threshold: 0.85 } });
|
||||
const config = loadConfig(tempDir);
|
||||
expect(config.autonomy.decision_confidence_threshold).toBe(0.85);
|
||||
expect(config.autonomy.level).toBe("full");
|
||||
@@ -93,7 +93,7 @@ describe("CI Config", () => {
|
||||
});
|
||||
|
||||
it("preserves nested objects that are not overridden", () => {
|
||||
initCI(tempDir, { git: { ...DEFAULT_CI_CONFIG.git, auto_push: true } });
|
||||
initCIAgent(tempDir, { git: { ...DEFAULT_CIAGENT_CONFIG.git, auto_push: true } });
|
||||
const config = loadConfig(tempDir);
|
||||
expect(config.git.auto_push).toBe(true);
|
||||
expect(config.git.auto_commit).toBe(true);
|
||||
@@ -101,7 +101,7 @@ describe("CI Config", () => {
|
||||
});
|
||||
|
||||
it("loads projects array from config", () => {
|
||||
initCI(tempDir, undefined, "task-api", "Task API");
|
||||
initCIAgent(tempDir, undefined, "task-api", "Task API");
|
||||
const config = loadConfig(tempDir);
|
||||
expect(config.projects).toHaveLength(1);
|
||||
expect(config.active_project).toBe("task-api");
|
||||
@@ -112,8 +112,8 @@ describe("CI Config", () => {
|
||||
it("saves and reloads config correctly", () => {
|
||||
ensureCIDir(tempDir);
|
||||
const customConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, level: "guided" as const },
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, level: "guided" as const },
|
||||
};
|
||||
saveConfig(tempDir, customConfig);
|
||||
const loaded = loadConfig(tempDir);
|
||||
@@ -123,7 +123,7 @@ describe("CI Config", () => {
|
||||
it("saves and reloads config with projects", () => {
|
||||
ensureCIDir(tempDir);
|
||||
const config = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
projects: [{ slug: "my-app", name: "My App", default: true }],
|
||||
active_project: "my-app",
|
||||
};
|
||||
@@ -134,27 +134,27 @@ describe("CI Config", () => {
|
||||
});
|
||||
});
|
||||
|
||||
describe("isCIInitialized", () => {
|
||||
describe("isCIAgentInitialized", () => {
|
||||
it("returns false for uninitialized directory", () => {
|
||||
expect(isCIInitialized(tempDir)).toBe(false);
|
||||
expect(isCIAgentInitialized(tempDir)).toBe(false);
|
||||
});
|
||||
|
||||
it("returns true after initCI", () => {
|
||||
initCI(tempDir);
|
||||
expect(isCIInitialized(tempDir)).toBe(true);
|
||||
it("returns true after initCIAgent", () => {
|
||||
initCIAgent(tempDir);
|
||||
expect(isCIAgentInitialized(tempDir)).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
describe("ensureCIDir", () => {
|
||||
it("creates .ci directory", () => {
|
||||
it("creates .ciagent directory", () => {
|
||||
ensureCIDir(tempDir);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent"))).toBe(true);
|
||||
});
|
||||
|
||||
it("is idempotent", () => {
|
||||
ensureCIDir(tempDir);
|
||||
ensureCIDir(tempDir);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ci"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(tempDir, ".ciagent"))).toBe(true);
|
||||
});
|
||||
});
|
||||
});
|
||||
+25
-25
@@ -1,11 +1,11 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import { CIConfig, DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { CIAgentConfig, DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
const CI_DIR = ".ci";
|
||||
const CI_DIR = ".ciagent";
|
||||
const CONFIG_FILE = "config.json";
|
||||
|
||||
export function getCIConfigPath(projectPath: string): string {
|
||||
export function getCIAgentConfigPath(projectPath: string): string {
|
||||
return path.join(projectPath, CI_DIR, CONFIG_FILE);
|
||||
}
|
||||
|
||||
@@ -20,7 +20,7 @@ export function ensureCIDir(projectPath: string): void {
|
||||
}
|
||||
}
|
||||
|
||||
function deepMerge(base: CIConfig, override: Record<string, unknown>): CIConfig {
|
||||
function deepMerge(base: CIAgentConfig, override: Record<string, unknown>): CIAgentConfig {
|
||||
const result = { ...base } as Record<string, unknown>;
|
||||
for (const key of Object.keys(override)) {
|
||||
const baseVal = result[key];
|
||||
@@ -30,43 +30,43 @@ function deepMerge(base: CIConfig, override: Record<string, unknown>): CIConfig
|
||||
overrideVal && typeof overrideVal === "object" && !Array.isArray(overrideVal)
|
||||
) {
|
||||
result[key] = deepMerge(
|
||||
baseVal as unknown as CIConfig,
|
||||
baseVal as unknown as CIAgentConfig,
|
||||
overrideVal as Record<string, unknown>
|
||||
) as unknown;
|
||||
} else if (overrideVal !== undefined) {
|
||||
result[key] = overrideVal;
|
||||
}
|
||||
}
|
||||
return result as unknown as CIConfig;
|
||||
return result as unknown as CIAgentConfig;
|
||||
}
|
||||
|
||||
export function loadConfig(projectPath: string): CIConfig {
|
||||
const configPath = getCIConfigPath(projectPath);
|
||||
export function loadConfig(projectPath: string): CIAgentConfig {
|
||||
const configPath = getCIAgentConfigPath(projectPath);
|
||||
if (!fs.existsSync(configPath)) {
|
||||
return { ...DEFAULT_CI_CONFIG };
|
||||
return { ...DEFAULT_CIAGENT_CONFIG };
|
||||
}
|
||||
const raw = fs.readFileSync(configPath, "utf-8");
|
||||
const parsed = JSON.parse(raw);
|
||||
return deepMerge(DEFAULT_CI_CONFIG, parsed);
|
||||
return deepMerge(DEFAULT_CIAGENT_CONFIG, parsed);
|
||||
}
|
||||
|
||||
export function saveConfig(projectPath: string, config: CIConfig): void {
|
||||
export function saveConfig(projectPath: string, config: CIAgentConfig): void {
|
||||
ensureCIDir(projectPath);
|
||||
const configPath = getCIConfigPath(projectPath);
|
||||
const configPath = getCIAgentConfigPath(projectPath);
|
||||
fs.writeFileSync(configPath, JSON.stringify(config, null, 2), "utf-8");
|
||||
}
|
||||
|
||||
export function isCIInitialized(projectPath: string): boolean {
|
||||
export function isCIAgentInitialized(projectPath: string): boolean {
|
||||
const ciDir = getCIDir(projectPath);
|
||||
const configPath = getCIConfigPath(projectPath);
|
||||
const configPath = getCIAgentConfigPath(projectPath);
|
||||
return fs.existsSync(ciDir) && fs.existsSync(configPath);
|
||||
}
|
||||
|
||||
export function initCI(projectPath: string, config?: Partial<CIConfig>, projectSlug?: string, projectName?: string): CIConfig {
|
||||
export function initCIAgent(projectPath: string, config?: Partial<CIAgentConfig>, projectSlug?: string, projectName?: string): CIAgentConfig {
|
||||
ensureCIDir(projectPath);
|
||||
|
||||
let projects = config?.projects || DEFAULT_CI_CONFIG.projects;
|
||||
let activeProject = config?.active_project || DEFAULT_CI_CONFIG.active_project;
|
||||
let projects = config?.projects || DEFAULT_CIAGENT_CONFIG.projects;
|
||||
let activeProject = config?.active_project || DEFAULT_CIAGENT_CONFIG.active_project;
|
||||
|
||||
if (projectSlug) {
|
||||
if (!projects.some((p) => p.slug === projectSlug)) {
|
||||
@@ -75,20 +75,20 @@ export function initCI(projectPath: string, config?: Partial<CIConfig>, projectS
|
||||
activeProject = projectSlug;
|
||||
}
|
||||
|
||||
const fullConfig: CIConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
const fullConfig: CIAgentConfig = {
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
...config,
|
||||
projects,
|
||||
active_project: activeProject,
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, ...config?.autonomy },
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, ...config?.autonomy },
|
||||
parallelization: {
|
||||
...DEFAULT_CI_CONFIG.parallelization,
|
||||
...DEFAULT_CIAGENT_CONFIG.parallelization,
|
||||
...config?.parallelization,
|
||||
},
|
||||
verification: { ...DEFAULT_CI_CONFIG.verification, ...config?.verification },
|
||||
security: { ...DEFAULT_CI_CONFIG.security, ...config?.security },
|
||||
git: { ...DEFAULT_CI_CONFIG.git, ...config?.git },
|
||||
backend: { ...DEFAULT_CI_CONFIG.backend, ...config?.backend },
|
||||
verification: { ...DEFAULT_CIAGENT_CONFIG.verification, ...config?.verification },
|
||||
security: { ...DEFAULT_CIAGENT_CONFIG.security, ...config?.security },
|
||||
git: { ...DEFAULT_CIAGENT_CONFIG.git, ...config?.git },
|
||||
backend: { ...DEFAULT_CIAGENT_CONFIG.backend, ...config?.backend },
|
||||
};
|
||||
saveConfig(projectPath, fullConfig);
|
||||
return fullConfig;
|
||||
|
||||
@@ -2,15 +2,15 @@ import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { DecisionEngine, DecisionInput } from "../core/decision-engine.js";
|
||||
import { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
describe("DecisionEngine", () => {
|
||||
let tempDir: string;
|
||||
let engine: DecisionEngine;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-decision-test-"));
|
||||
engine = new DecisionEngine(DEFAULT_CI_CONFIG, tempDir);
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-decision-test-"));
|
||||
engine = new DecisionEngine(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
@@ -106,8 +106,8 @@ describe("DecisionEngine", () => {
|
||||
|
||||
it("escalates if threshold is raised above 0.7", () => {
|
||||
const strictConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, decision_confidence_threshold: 0.8 },
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, decision_confidence_threshold: 0.8 },
|
||||
};
|
||||
const strictEngine = new DecisionEngine(strictConfig, tempDir);
|
||||
const result = strictEngine.makeMediumConfidenceDecision(
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
import { execSync } from "node:child_process";
|
||||
import { Decision, DecisionCategory, Alternative, confidenceToLevel } from "../types/decisions.js";
|
||||
import { CIConfig } from "../types/config.js";
|
||||
import { CIAgentConfig } from "../types/config.js";
|
||||
import { CommitBuilder, DecisionCommitInput } from "./commit-builder.js";
|
||||
import { CommitDecision } from "../types/commit-meta.js";
|
||||
|
||||
@@ -22,13 +22,13 @@ export interface DecisionResult {
|
||||
}
|
||||
|
||||
export class DecisionEngine {
|
||||
private config: CIConfig;
|
||||
private config: CIAgentConfig;
|
||||
private projectPath: string;
|
||||
private currentPhase: number;
|
||||
private currentMilestone: string;
|
||||
private decisionCounter: number;
|
||||
|
||||
constructor(config: CIConfig, projectPath: string, milestone: string = "v1.0") {
|
||||
constructor(config: CIAgentConfig, projectPath: string, milestone: string = "v1.0") {
|
||||
this.config = config;
|
||||
this.projectPath = projectPath;
|
||||
this.currentPhase = 0;
|
||||
|
||||
@@ -2,16 +2,16 @@ import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { ErrorRecovery } from "../core/error-recovery.js";
|
||||
import { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
describe("ErrorRecovery", () => {
|
||||
let tempDir: string;
|
||||
let recovery: ErrorRecovery;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-recovery-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ci"), { recursive: true });
|
||||
recovery = new ErrorRecovery(DEFAULT_CI_CONFIG, tempDir);
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-recovery-test-"));
|
||||
fs.mkdirSync(path.join(tempDir, ".ciagent"), { recursive: true });
|
||||
recovery = new ErrorRecovery(DEFAULT_CIAGENT_CONFIG, tempDir);
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
import { execSync } from "node:child_process";
|
||||
import { CIConfig } from "../types/config.js";
|
||||
import { CIAgentConfig } from "../types/config.js";
|
||||
|
||||
export interface RetryConfig {
|
||||
max_retries: number;
|
||||
@@ -15,11 +15,11 @@ export interface RecoveryResult {
|
||||
}
|
||||
|
||||
export class ErrorRecovery {
|
||||
private config: CIConfig;
|
||||
private config: CIAgentConfig;
|
||||
private projectPath: string;
|
||||
private revisionCount: number;
|
||||
|
||||
constructor(config: CIConfig, projectPath: string) {
|
||||
constructor(config: CIAgentConfig, projectPath: string) {
|
||||
this.config = config;
|
||||
this.projectPath = projectPath;
|
||||
this.revisionCount = 0;
|
||||
|
||||
@@ -2,17 +2,17 @@ import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { EscalationProtocol, EscalationInput } from "../core/escalation.js";
|
||||
import { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
describe("EscalationProtocol", () => {
|
||||
let tempDir: string;
|
||||
let protocol: EscalationProtocol;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-escalation-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-escalation-test-"));
|
||||
const noAutoCommitConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
git: { ...DEFAULT_CI_CONFIG.git, auto_commit: false },
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
git: { ...DEFAULT_CIAGENT_CONFIG.git, auto_commit: false },
|
||||
};
|
||||
protocol = new EscalationProtocol(noAutoCommitConfig, tempDir);
|
||||
});
|
||||
|
||||
+25
-4
@@ -6,7 +6,7 @@ import {
|
||||
EscalationResolution,
|
||||
ESCALATION_TYPES,
|
||||
} from "../types/escalation.js";
|
||||
import { CIConfig } from "../types/config.js";
|
||||
import { CIAgentConfig } from "../types/config.js";
|
||||
import { CommitBuilder, EscalationCommitInput } from "./commit-builder.js";
|
||||
import { CommitEscalation } from "../types/commit-meta.js";
|
||||
|
||||
@@ -22,16 +22,17 @@ export interface EscalationInput {
|
||||
}
|
||||
|
||||
export class EscalationProtocol {
|
||||
private config: CIConfig;
|
||||
private config: CIAgentConfig;
|
||||
private projectPath: string;
|
||||
private currentMilestone: string;
|
||||
private counter: number;
|
||||
private pendingEscalations: Map<string, Escalation>;
|
||||
private timeoutCallback: (escalation: Escalation, chosenOption: string) => void;
|
||||
private timers: NodeJS.Timeout[];
|
||||
private timerEscalationMap: Map<NodeJS.Timeout, string>;
|
||||
|
||||
constructor(
|
||||
config: CIConfig,
|
||||
config: CIAgentConfig,
|
||||
projectPath: string,
|
||||
milestone: string = "v1.0",
|
||||
timeoutCallback: (escalation: Escalation, chosenOption: string) => void = () => {}
|
||||
@@ -43,6 +44,7 @@ export class EscalationProtocol {
|
||||
this.pendingEscalations = new Map();
|
||||
this.timeoutCallback = timeoutCallback;
|
||||
this.timers = [];
|
||||
this.timerEscalationMap = new Map();
|
||||
}
|
||||
|
||||
setMilestone(milestone: string): void {
|
||||
@@ -64,7 +66,7 @@ export class EscalationProtocol {
|
||||
options: input.options,
|
||||
default_option_id: input.default_option_id,
|
||||
resolution: "pending",
|
||||
audit_file: `.ci/audit/deprecated`,
|
||||
audit_file: `.ciagent/audit/deprecated`,
|
||||
};
|
||||
|
||||
this.pendingEscalations.set(id, escalation);
|
||||
@@ -102,6 +104,16 @@ export class EscalationProtocol {
|
||||
const escalation = this.pendingEscalations.get(escalationId);
|
||||
if (!escalation) return null;
|
||||
|
||||
for (let i = this.timers.length - 1; i >= 0; i--) {
|
||||
const timer = this.timers[i];
|
||||
const mappedId = this.timerEscalationMap.get(timer);
|
||||
if (mappedId === escalationId) {
|
||||
clearTimeout(timer);
|
||||
this.timerEscalationMap.delete(timer);
|
||||
this.timers.splice(i, 1);
|
||||
}
|
||||
}
|
||||
|
||||
escalation.resolution = resolution;
|
||||
escalation.resolved_at = new Date().toISOString();
|
||||
escalation.resolution_detail = `Chose option: ${chosenOptionId}`;
|
||||
@@ -139,11 +151,16 @@ export class EscalationProtocol {
|
||||
clearAllTimers(): void {
|
||||
for (const timer of this.timers) {
|
||||
clearTimeout(timer);
|
||||
this.timerEscalationMap.delete(timer);
|
||||
}
|
||||
this.timers = [];
|
||||
this.pendingEscalations.clear();
|
||||
}
|
||||
|
||||
dispose(): void {
|
||||
this.clearAllTimers();
|
||||
}
|
||||
|
||||
formatEscalation(escalation: Escalation): string {
|
||||
const lines: string[] = [
|
||||
`⚠️ ESCALATION [${escalation.id}]`,
|
||||
@@ -200,9 +217,13 @@ export class EscalationProtocol {
|
||||
escalation.resolved_at = new Date().toISOString();
|
||||
escalation.resolution_detail = `Auto-proceeded with default: ${escalation.default_option_id}`;
|
||||
this.pendingEscalations.delete(escalation.id);
|
||||
this.timerEscalationMap.delete(timer);
|
||||
const idx = this.timers.indexOf(timer);
|
||||
if (idx >= 0) this.timers.splice(idx, 1);
|
||||
this.timeoutCallback(escalation, escalation.default_option_id);
|
||||
}
|
||||
}, timeout);
|
||||
this.timers.push(timer);
|
||||
this.timerEscalationMap.set(timer, escalation.id);
|
||||
}
|
||||
}
|
||||
@@ -5,7 +5,7 @@ import * as fs from "node:fs";
|
||||
import { GitBranch } from "../core/git-branch.js";
|
||||
|
||||
function createTempRepo(): string {
|
||||
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-branch-test-"));
|
||||
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-branch-test-"));
|
||||
execSync("git init", { cwd: dir, stdio: "pipe" });
|
||||
execSync('git config user.email "test@test.com"', { cwd: dir, stdio: "pipe" });
|
||||
execSync('git config user.name "Test"', { cwd: dir, stdio: "pipe" });
|
||||
|
||||
@@ -5,7 +5,7 @@ import * as fs from "node:fs";
|
||||
import { GitContext } from "../core/git-context.js";
|
||||
|
||||
function createTempRepo(): string {
|
||||
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-test-"));
|
||||
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-test-"));
|
||||
execSync("git init", { cwd: dir, stdio: "pipe" });
|
||||
execSync('git config user.email "test@test.com"', { cwd: dir, stdio: "pipe" });
|
||||
execSync('git config user.name "Test"', { cwd: dir, stdio: "pipe" });
|
||||
@@ -41,7 +41,7 @@ describe("GitContext", () => {
|
||||
});
|
||||
|
||||
it("returns false for non-git directory", () => {
|
||||
const nonGit = fs.mkdtempSync(path.join(os.tmpdir(), "ci-nongit-"));
|
||||
const nonGit = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-nongit-"));
|
||||
const ctx = new GitContext(nonGit);
|
||||
expect(ctx.isGitRepo()).toBe(false);
|
||||
cleanup(nonGit);
|
||||
|
||||
+10
-10
@@ -1,7 +1,7 @@
|
||||
import { execSync } from "node:child_process";
|
||||
import {
|
||||
ParsedCiCommit,
|
||||
CiMetadata,
|
||||
ParsedCIAgentCommit,
|
||||
CIAgentMetadata,
|
||||
CommitDecision,
|
||||
} from "../types/commit-meta.js";
|
||||
import { parseCommitMessage } from "./commit-parser.js";
|
||||
@@ -16,7 +16,7 @@ export interface ProjectState {
|
||||
phasesCompleted: number[];
|
||||
phaseBranches: BranchInfo[];
|
||||
milestoneBranches: string[];
|
||||
lastCommit: ParsedCiCommit | null;
|
||||
lastCommit: ParsedCIAgentCommit | null;
|
||||
}
|
||||
|
||||
export interface BranchInfo {
|
||||
@@ -69,13 +69,13 @@ export class GitContext {
|
||||
return this.git("rev-parse --abbrev-ref HEAD");
|
||||
}
|
||||
|
||||
getRecentCommits(count: number = 20): ParsedCiCommit[] {
|
||||
getRecentCommits(count: number = 20): ParsedCIAgentCommit[] {
|
||||
const format = "%H%x00%s%x00%B%x01";
|
||||
const raw = this.git(`log --max-count=${count} --format="${format}"`);
|
||||
|
||||
if (!raw) return [];
|
||||
|
||||
const commits: ParsedCiCommit[] = [];
|
||||
const commits: ParsedCIAgentCommit[] = [];
|
||||
const entries = raw.split("\x01").filter(Boolean);
|
||||
|
||||
for (const entry of entries) {
|
||||
@@ -93,7 +93,7 @@ export class GitContext {
|
||||
return commits;
|
||||
}
|
||||
|
||||
getLatestCiCommit(): ParsedCiCommit | null {
|
||||
getLatestCiCommit(): ParsedCIAgentCommit | null {
|
||||
const commits = this.getRecentCommits(1);
|
||||
return commits.length > 0 ? commits[0] : null;
|
||||
}
|
||||
@@ -207,7 +207,7 @@ export class GitContext {
|
||||
return decisions;
|
||||
}
|
||||
|
||||
getDecisionsFromCommits(commits: ParsedCiCommit[], phase?: number): CommitDecision[] {
|
||||
getDecisionsFromCommits(commits: ParsedCIAgentCommit[], phase?: number): CommitDecision[] {
|
||||
const decisions: CommitDecision[] = [];
|
||||
for (const commit of commits) {
|
||||
if (commit.ci?.decisions) {
|
||||
@@ -300,20 +300,20 @@ export class GitContext {
|
||||
};
|
||||
}
|
||||
|
||||
getCommitsForPhase(phase: number): ParsedCiCommit[] {
|
||||
getCommitsForPhase(phase: number): ParsedCIAgentCommit[] {
|
||||
const commits = this.getRecentCommits(200);
|
||||
return commits.filter(
|
||||
(c) => c.scope === `P${String(phase).padStart(2, "0")}` || c.ci?.phase === phase
|
||||
);
|
||||
}
|
||||
|
||||
getCommitsForBranch(branch: string): ParsedCiCommit[] {
|
||||
getCommitsForBranch(branch: string): ParsedCIAgentCommit[] {
|
||||
const format = "%H%x00%s%x00%B%x01";
|
||||
const raw = this.git(`log ${branch} --max-count=100 --format="${format}"`);
|
||||
|
||||
if (!raw) return [];
|
||||
|
||||
const commits: ParsedCiCommit[] = [];
|
||||
const commits: ParsedCIAgentCommit[] = [];
|
||||
const entries = raw.split("\x01").filter(Boolean);
|
||||
|
||||
for (const entry of entries) {
|
||||
|
||||
@@ -0,0 +1,191 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { GiteaClient, generateReleaseNotes, GiteaReleaseConfig } from "../core/gitea.js";
|
||||
|
||||
const defaultConfig: GiteaReleaseConfig = {
|
||||
baseUrl: "https://git.example.com",
|
||||
token: "test-token-123",
|
||||
owner: "testorg",
|
||||
repo: "testrepo",
|
||||
};
|
||||
|
||||
function makeReleaseResponse(overrides: Partial<{
|
||||
id: number;
|
||||
tag_name: string;
|
||||
name: string;
|
||||
body: string;
|
||||
url: string;
|
||||
html_url: string;
|
||||
draft: boolean;
|
||||
prerelease: boolean;
|
||||
}> = {}): Record<string, unknown> {
|
||||
return {
|
||||
id: overrides.id ?? 1,
|
||||
tag_name: overrides.tag_name ?? "v1.0.0",
|
||||
name: overrides.name ?? "v1.0.0",
|
||||
body: overrides.body ?? "Release notes",
|
||||
url: overrides.url ?? "https://git.example.com/api/v1/repos/testorg/testrepo/releases/1",
|
||||
html_url: overrides.html_url ?? "https://git.example.com/testorg/testrepo/releases/tag/v1.0.0",
|
||||
draft: overrides.draft ?? false,
|
||||
prerelease: overrides.prerelease ?? false,
|
||||
};
|
||||
}
|
||||
|
||||
describe("GiteaClient", () => {
|
||||
let originalFetch: typeof globalThis.fetch;
|
||||
|
||||
beforeEach(() => {
|
||||
originalFetch = globalThis.fetch;
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
globalThis.fetch = originalFetch;
|
||||
});
|
||||
|
||||
describe("createRelease", () => {
|
||||
it("creates a release via POST", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: true,
|
||||
json: async () => makeReleaseResponse({ tag_name: "v1.0.0", name: "v1.0.0" }),
|
||||
});
|
||||
|
||||
const release = await client.createRelease({
|
||||
tag_name: "v1.0.0",
|
||||
name: "v1.0.0",
|
||||
body: "Initial release",
|
||||
});
|
||||
|
||||
expect(release.tag_name).toBe("v1.0.0");
|
||||
expect(globalThis.fetch).toHaveBeenCalledTimes(1);
|
||||
const call = (globalThis.fetch as jest.Mock).mock.calls[0];
|
||||
expect(call[0]).toContain("/releases");
|
||||
expect(call[1].method).toBe("POST");
|
||||
expect(call[1].headers.Authorization).toBe("token test-token-123");
|
||||
});
|
||||
|
||||
it("throws on non-ok response", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: false,
|
||||
status: 409,
|
||||
text: async () => "Conflict: tag already exists",
|
||||
});
|
||||
|
||||
await expect(client.createRelease({
|
||||
tag_name: "v1.0.0",
|
||||
name: "v1.0.0",
|
||||
body: "",
|
||||
})).rejects.toThrow("Gitea API error: 409");
|
||||
});
|
||||
});
|
||||
|
||||
describe("listReleases", () => {
|
||||
it("lists releases via GET", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: true,
|
||||
json: async () => [
|
||||
makeReleaseResponse({ id: 1, tag_name: "v1.0.0" }),
|
||||
makeReleaseResponse({ id: 2, tag_name: "v1.1.0" }),
|
||||
],
|
||||
});
|
||||
|
||||
const releases = await client.listReleases();
|
||||
expect(releases).toHaveLength(2);
|
||||
expect(releases[0].tag_name).toBe("v1.0.0");
|
||||
expect(releases[1].tag_name).toBe("v1.1.0");
|
||||
});
|
||||
|
||||
it("throws on non-ok response", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: false,
|
||||
status: 500,
|
||||
});
|
||||
|
||||
await expect(client.listReleases()).rejects.toThrow("Gitea API error: 500");
|
||||
});
|
||||
});
|
||||
|
||||
describe("getReleaseByTag", () => {
|
||||
it("returns release when found", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: true,
|
||||
status: 200,
|
||||
json: async () => makeReleaseResponse({ tag_name: "v1.0.0" }),
|
||||
});
|
||||
|
||||
const release = await client.getReleaseByTag("v1.0.0");
|
||||
expect(release).not.toBeNull();
|
||||
expect(release!.tag_name).toBe("v1.0.0");
|
||||
});
|
||||
|
||||
it("returns null on 404", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: false,
|
||||
status: 404,
|
||||
});
|
||||
|
||||
const release = await client.getReleaseByTag("v0.0.0");
|
||||
expect(release).toBeNull();
|
||||
});
|
||||
|
||||
it("throws on other non-ok status", async () => {
|
||||
const client = new GiteaClient(defaultConfig);
|
||||
globalThis.fetch = jest.fn().mockResolvedValue({
|
||||
ok: false,
|
||||
status: 500,
|
||||
});
|
||||
|
||||
await expect(client.getReleaseByTag("v1.0.0")).rejects.toThrow("Gitea API error: 500");
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe("generateReleaseNotes", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-gitea-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
});
|
||||
|
||||
it("parses git log into categorized sections", () => {
|
||||
const gitDir = path.join(dir, "repo");
|
||||
fs.mkdirSync(gitDir, { recursive: true });
|
||||
|
||||
const { execSync } = require("node:child_process");
|
||||
execSync("git init", { cwd: gitDir, stdio: "pipe" });
|
||||
execSync('git config user.email "test@test.com"', { cwd: gitDir, stdio: "pipe" });
|
||||
execSync('git config user.name "Test"', { cwd: gitDir, stdio: "pipe" });
|
||||
|
||||
fs.writeFileSync(path.join(gitDir, "file1.txt"), "hello");
|
||||
execSync("git add -A", { cwd: gitDir, stdio: "pipe" });
|
||||
execSync('git commit -m "feat: add authentication"', { cwd: gitDir, stdio: "pipe" });
|
||||
|
||||
fs.writeFileSync(path.join(gitDir, "file2.txt"), "world");
|
||||
execSync("git add -A", { cwd: gitDir, stdio: "pipe" });
|
||||
execSync('git commit -m "fix: resolve login bug"', { cwd: gitDir, stdio: "pipe" });
|
||||
|
||||
execSync("git tag v1.0.0", { cwd: gitDir, stdio: "pipe" });
|
||||
|
||||
const notes = generateReleaseNotes(gitDir, null, "v1.0.0");
|
||||
expect(notes).toContain("New Features");
|
||||
expect(notes).toContain("add authentication");
|
||||
expect(notes).toContain("Bug Fixes");
|
||||
expect(notes).toContain("resolve login bug");
|
||||
});
|
||||
|
||||
it("returns no-commits message when no commits found", () => {
|
||||
const nonExistent = path.join(dir, "nonexistent");
|
||||
const notes = generateReleaseNotes(nonExistent, null, "v0.0.0");
|
||||
expect(notes).toContain("No commits found");
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,170 @@
|
||||
import { execSync } from "node:child_process";
|
||||
|
||||
export interface GiteaReleaseConfig {
|
||||
baseUrl: string;
|
||||
token: string;
|
||||
owner: string;
|
||||
repo: string;
|
||||
}
|
||||
|
||||
export interface GiteaRelease {
|
||||
id: number;
|
||||
tag_name: string;
|
||||
name: string;
|
||||
body: string;
|
||||
url: string;
|
||||
html_url: string;
|
||||
draft: boolean;
|
||||
prerelease: boolean;
|
||||
}
|
||||
|
||||
export class GiteaClient {
|
||||
private config: GiteaReleaseConfig;
|
||||
|
||||
constructor(config: GiteaReleaseConfig) {
|
||||
this.config = config;
|
||||
}
|
||||
|
||||
async createRelease(params: {
|
||||
tag_name: string;
|
||||
name: string;
|
||||
body: string;
|
||||
draft?: boolean;
|
||||
prerelease?: boolean;
|
||||
}): Promise<GiteaRelease> {
|
||||
const url = `${this.config.baseUrl}/api/v1/repos/${this.config.owner}/${this.config.repo}/releases`;
|
||||
const response = await fetch(url, {
|
||||
method: "POST",
|
||||
headers: {
|
||||
"Authorization": `token ${this.config.token}`,
|
||||
"Content-Type": "application/json",
|
||||
},
|
||||
body: JSON.stringify({
|
||||
tag_name: params.tag_name,
|
||||
name: params.name,
|
||||
body: params.body,
|
||||
draft: params.draft ?? false,
|
||||
prerelease: params.prerelease ?? false,
|
||||
}),
|
||||
});
|
||||
|
||||
if (!response.ok) {
|
||||
const text = await response.text();
|
||||
throw new Error(`Gitea API error: ${response.status} ${text}`);
|
||||
}
|
||||
|
||||
return response.json() as Promise<GiteaRelease>;
|
||||
}
|
||||
|
||||
async listReleases(): Promise<GiteaRelease[]> {
|
||||
const url = `${this.config.baseUrl}/api/v1/repos/${this.config.owner}/${this.config.repo}/releases`;
|
||||
const response = await fetch(url, {
|
||||
method: "GET",
|
||||
headers: {
|
||||
"Authorization": `token ${this.config.token}`,
|
||||
},
|
||||
});
|
||||
|
||||
if (!response.ok) {
|
||||
throw new Error(`Gitea API error: ${response.status}`);
|
||||
}
|
||||
|
||||
return response.json() as Promise<GiteaRelease[]>;
|
||||
}
|
||||
|
||||
async getReleaseByTag(tag: string): Promise<GiteaRelease | null> {
|
||||
const url = `${this.config.baseUrl}/api/v1/repos/${this.config.owner}/${this.config.repo}/releases/tags/${tag}`;
|
||||
const response = await fetch(url, {
|
||||
method: "GET",
|
||||
headers: {
|
||||
"Authorization": `token ${this.config.token}`,
|
||||
},
|
||||
});
|
||||
|
||||
if (response.status === 404) {
|
||||
return null;
|
||||
}
|
||||
|
||||
if (!response.ok) {
|
||||
throw new Error(`Gitea API error: ${response.status}`);
|
||||
}
|
||||
|
||||
return response.json() as Promise<GiteaRelease>;
|
||||
}
|
||||
}
|
||||
|
||||
export function generateReleaseNotes(projectPath: string, fromTag: string | null, toTag: string): string {
|
||||
let gitLogCmd: string;
|
||||
if (fromTag) {
|
||||
gitLogCmd = `git log ${fromTag}..${toTag} --oneline`;
|
||||
} else {
|
||||
gitLogCmd = `git log ${toTag} --oneline`;
|
||||
}
|
||||
|
||||
let logOutput: string;
|
||||
try {
|
||||
logOutput = execSync(gitLogCmd, { cwd: projectPath, encoding: "utf-8" }).trim();
|
||||
} catch {
|
||||
return `## What's Changed\n\nNo commits found between ${fromTag || "beginning"} and ${toTag}.\n`;
|
||||
}
|
||||
|
||||
if (!logOutput) {
|
||||
return `## What's Changed\n\nNo commits found between ${fromTag || "beginning"} and ${toTag}.\n`;
|
||||
}
|
||||
|
||||
const lines = logOutput.split("\n").filter(Boolean);
|
||||
|
||||
const featCommits: string[] = [];
|
||||
const fixCommits: string[] = [];
|
||||
const testCommits: string[] = [];
|
||||
const otherCommits: string[] = [];
|
||||
|
||||
for (const line of lines) {
|
||||
const subject = line.replace(/^[a-f0-9]+\s+/, "");
|
||||
if (/^feat/i.test(subject)) {
|
||||
featCommits.push(subject);
|
||||
} else if (/^fix/i.test(subject)) {
|
||||
fixCommits.push(subject);
|
||||
} else if (/^test/i.test(subject)) {
|
||||
testCommits.push(subject);
|
||||
} else {
|
||||
otherCommits.push(subject);
|
||||
}
|
||||
}
|
||||
|
||||
const sections: string[] = [];
|
||||
|
||||
if (featCommits.length > 0) {
|
||||
sections.push("### New Features\n");
|
||||
for (const c of featCommits) {
|
||||
sections.push(`- ${c}`);
|
||||
}
|
||||
sections.push("");
|
||||
}
|
||||
|
||||
if (fixCommits.length > 0) {
|
||||
sections.push("### Bug Fixes\n");
|
||||
for (const c of fixCommits) {
|
||||
sections.push(`- ${c}`);
|
||||
}
|
||||
sections.push("");
|
||||
}
|
||||
|
||||
if (testCommits.length > 0) {
|
||||
sections.push("### Tests\n");
|
||||
for (const c of testCommits) {
|
||||
sections.push(`- ${c}`);
|
||||
}
|
||||
sections.push("");
|
||||
}
|
||||
|
||||
if (otherCommits.length > 0) {
|
||||
sections.push("### Other Changes\n");
|
||||
for (const c of otherCommits) {
|
||||
sections.push(`- ${c}`);
|
||||
}
|
||||
sections.push("");
|
||||
}
|
||||
|
||||
return `## What's Changed\n\n${sections.join("\n")}`;
|
||||
}
|
||||
+7
-5
@@ -1,12 +1,14 @@
|
||||
export { initCI, loadConfig, saveConfig, isCIInitialized, getCIConfigPath, getCIDir, ensureCIDir } from "./config.js";
|
||||
export { initCIAgent, loadConfig, saveConfig, isCIAgentInitialized, getCIAgentConfigPath, getCIDir, ensureCIDir } from "./config.js";
|
||||
export { DecisionEngine } from "./decision-engine.js";
|
||||
export { EscalationProtocol } from "./escalation.js";
|
||||
export { ClarifyPhase } from "./clarify.js";
|
||||
export { CiFiles } from "./ci-files.js";
|
||||
export { CIAgentFiles } from "./ciagent-files.js";
|
||||
export { ErrorRecovery } from "./error-recovery.js";
|
||||
export { GitContext } from "./git-context.js";
|
||||
export { GitBranch } from "./git-branch.js";
|
||||
export { CommitBuilder } from "./commit-builder.js";
|
||||
export { extractCiBlock, parseCiBlock, parseCommitMessage } from "./commit-parser.js";
|
||||
export type { CIConfig } from "../types/config.js";
|
||||
export { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
export { extractCIAgentBlock, parseCIAgentBlock, parseCommitMessage } from "./commit-parser.js";
|
||||
export { GiteaClient, generateReleaseNotes } from "./gitea.js";
|
||||
export type { GiteaReleaseConfig, GiteaRelease } from "./gitea.js";
|
||||
export type { CIAgentConfig } from "../types/config.js";
|
||||
export { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
@@ -0,0 +1,171 @@
|
||||
import * as fs from "node:fs";
|
||||
import * as path from "node:path";
|
||||
import * as os from "node:os";
|
||||
import { CIAgentFiles, ProjectEntry } from "../core/ciagent-files.js";
|
||||
import { initCIAgent, loadConfig, saveConfig } from "../core/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
function createTempDir(): string {
|
||||
return fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-multiproject-test-"));
|
||||
}
|
||||
|
||||
function cleanup(dir: string): void {
|
||||
fs.rmSync(dir, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
describe("Multi-project CIAgentFiles operations", () => {
|
||||
let dir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
dir = createTempDir();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
cleanup(dir);
|
||||
});
|
||||
|
||||
describe("--project flag behavior via CIAgentFiles", () => {
|
||||
it("sets active project via setActiveProject", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.addProject("task-api", "Task API");
|
||||
ciFiles.addProject("auth-svc", "Auth Service");
|
||||
|
||||
ciFiles.setActiveProject("auth-svc");
|
||||
expect(ciFiles.getActiveProject()).toBe("auth-svc");
|
||||
});
|
||||
|
||||
it("lists all added projects", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.addProject("task-api", "Task API");
|
||||
ciFiles.addProject("auth-svc", "Auth Service");
|
||||
|
||||
const projects = ciFiles.listProjects();
|
||||
expect(projects.length).toBeGreaterThanOrEqual(2);
|
||||
const slugs = projects.map(p => p.slug);
|
||||
expect(slugs).toContain("task-api");
|
||||
expect(slugs).toContain("auth-svc");
|
||||
});
|
||||
|
||||
it("addProject does not duplicate existing slug", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.addProject("task-api", "Task API");
|
||||
ciFiles.addProject("task-api", "Task API V2");
|
||||
|
||||
const projects = ciFiles.listProjects();
|
||||
const taskApiProjects = projects.filter(p => p.slug === "task-api");
|
||||
expect(taskApiProjects.length).toBe(1);
|
||||
});
|
||||
|
||||
it("defaults to empty string when no active project set", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
expect(ciFiles.getActiveProject()).toBe("");
|
||||
});
|
||||
|
||||
it("isMultiProject returns false for single or no projects", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
expect(ciFiles.isMultiProject()).toBe(false);
|
||||
});
|
||||
|
||||
it("isMultiProject returns true when projects exist in config", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.addProject("task-api", "Task API");
|
||||
ciFiles.addProject("auth-svc", "Auth Service");
|
||||
expect(ciFiles.isMultiProject()).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
describe("config-level project operations", () => {
|
||||
it("initCIAgent with slug adds project to config", () => {
|
||||
const config = initCIAgent(dir, undefined, "task-api", "Task API");
|
||||
expect(config.projects).toHaveLength(1);
|
||||
expect(config.active_project).toBe("task-api");
|
||||
});
|
||||
|
||||
it("--project override sets active_project in config", () => {
|
||||
initCIAgent(dir, undefined, "task-api", "Task API");
|
||||
const config = loadConfig(dir);
|
||||
config.active_project = "task-api";
|
||||
config.projects = [
|
||||
{ slug: "task-api", name: "Task API", default: true },
|
||||
{ slug: "auth-svc", name: "Auth Service" },
|
||||
];
|
||||
saveConfig(dir, config);
|
||||
|
||||
const loaded = loadConfig(dir);
|
||||
expect(loaded.active_project).toBe("task-api");
|
||||
expect(loaded.projects).toHaveLength(2);
|
||||
});
|
||||
|
||||
it("setActiveProject persists to config", () => {
|
||||
initCIAgent(dir, undefined, "task-api", "Task API");
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.addProject("auth-svc", "Auth Service");
|
||||
ciFiles.setActiveProject("auth-svc");
|
||||
|
||||
const config = loadConfig(dir);
|
||||
expect(config.active_project).toBe("auth-svc");
|
||||
});
|
||||
});
|
||||
|
||||
describe("project slug and directory structure", () => {
|
||||
it("multi-project mode uses .ciagent/<slug>/ subdirectory", () => {
|
||||
const ciFiles = new CIAgentFiles(dir, "task-api");
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.ensureProjectDir();
|
||||
|
||||
const projectDir = path.join(dir, ".ciagent", "task-api");
|
||||
expect(fs.existsSync(projectDir)).toBe(true);
|
||||
});
|
||||
|
||||
it("single-project mode uses .ciagent/ directly", () => {
|
||||
const ciFiles = new CIAgentFiles(dir);
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.ensureProjectDir();
|
||||
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent"))).toBe(true);
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "task-api"))).toBe(false);
|
||||
});
|
||||
|
||||
it("writeProjectMd writes to project subdirectory in multi-project", () => {
|
||||
const ciFiles = new CIAgentFiles(dir, "task-api");
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.ensureProjectDir();
|
||||
|
||||
ciFiles.writeProjectMd({
|
||||
name: "Task API",
|
||||
coreValue: "Manage tasks",
|
||||
requirements: { validated: [], active: ["Task CRUD"], outOfScope: [] },
|
||||
constraints: ["Node.js"],
|
||||
context: "REST API",
|
||||
keyDecisions: [],
|
||||
}, "test write");
|
||||
|
||||
expect(fs.existsSync(path.join(dir, ".ciagent", "task-api", "PROJECT.md"))).toBe(true);
|
||||
});
|
||||
|
||||
it("readProjectMd reads from project subdirectory in multi-project", () => {
|
||||
const ciFiles = new CIAgentFiles(dir, "task-api");
|
||||
ciFiles.ensureCIDir();
|
||||
ciFiles.ensureProjectDir();
|
||||
|
||||
ciFiles.writeProjectMd({
|
||||
name: "Task API",
|
||||
coreValue: "Manage tasks",
|
||||
requirements: { validated: [], active: [], outOfScope: [] },
|
||||
constraints: [],
|
||||
context: "",
|
||||
keyDecisions: [],
|
||||
}, "test write");
|
||||
|
||||
const projectMd = ciFiles.readProjectMd();
|
||||
expect(projectMd).not.toBeNull();
|
||||
expect(projectMd!.name).toBe("Task API");
|
||||
});
|
||||
});
|
||||
});
|
||||
+11
-7
@@ -2,20 +2,23 @@ export { OrchestratorAgent } from "./agents/orchestrator.js";
|
||||
export { DecisionEngine } from "./core/decision-engine.js";
|
||||
export { EscalationProtocol } from "./core/escalation.js";
|
||||
export { ClarifyPhase } from "./core/clarify.js";
|
||||
export { CiFiles } from "./core/ci-files.js";
|
||||
export { CIAgentFiles } from "./core/ciagent-files.js";
|
||||
export { ErrorRecovery } from "./core/error-recovery.js";
|
||||
export { GitContext } from "./core/git-context.js";
|
||||
export { GitBranch } from "./core/git-branch.js";
|
||||
export { CommitBuilder } from "./core/commit-builder.js";
|
||||
export { extractCiBlock, parseCiBlock, parseCommitMessage } from "./core/commit-parser.js";
|
||||
export { extractCIAgentBlock, parseCIAgentBlock, parseCommitMessage } from "./core/commit-parser.js";
|
||||
export { GiteaClient, generateReleaseNotes } from "./core/gitea.js";
|
||||
export { VerificationPipeline } from "./verification/index.js";
|
||||
export { StructuralVerification } from "./verification/structural.js";
|
||||
export { BehavioralVerification } from "./verification/behavioral.js";
|
||||
export { SecurityVerification } from "./verification/security.js";
|
||||
export { QualityVerification } from "./verification/quality.js";
|
||||
export { getAgent, getAvailableAgents } from "./agents/index.js";
|
||||
export { initCI, loadConfig, saveConfig, isCIInitialized } from "./core/config.js";
|
||||
export { DEFAULT_CI_CONFIG } from "./types/config.js";
|
||||
export type { PlannerResult } from "./agents/planner.js";
|
||||
export type { ExecutorResult } from "./agents/executor.js";
|
||||
export { initCIAgent, loadConfig, saveConfig, isCIAgentInitialized } from "./core/config.js";
|
||||
export { DEFAULT_CIAGENT_CONFIG } from "./types/config.js";
|
||||
export { confidenceToLevel, shouldEscalate } from "./types/decisions.js";
|
||||
export { ESCALATION_TYPES } from "./types/escalation.js";
|
||||
export { createClarifyQuestion } from "./types/clarify.js";
|
||||
@@ -28,7 +31,7 @@ export { OllamaLocalBackend } from "./backends/ollama-local.js";
|
||||
export { OllamaCloudBackend } from "./backends/ollama-cloud.js";
|
||||
export { ToolRegistry } from "./backends/tool-registry.js";
|
||||
|
||||
export type { CIConfig, AutonomyLevel, ModelProfile } from "./types/config.js";
|
||||
export type { CIAgentConfig, AutonomyLevel, ModelProfile, GiteaConfig } from "./types/config.js";
|
||||
export type { Decision, DecisionCategory } from "./types/decisions.js";
|
||||
export type { Escalation, EscalationType } from "./types/escalation.js";
|
||||
export type { PipelineState, PhaseResult, OrchestratorResult } from "./types/pipeline.js";
|
||||
@@ -38,9 +41,10 @@ export type { AgentContext, AgentResult } from "./agents/base.js";
|
||||
export type { LayeredVerificationResult } from "./verification/index.js";
|
||||
export type { VerificationResult, VerificationCheck } from "./verification/types.js";
|
||||
export type { AgentName } from "./types/config.js";
|
||||
export type { CiMetadata, ParsedCiCommit, CommitType, CommitScope, CommitDecision, CommitEscalation, CommitRequirements, CommitCompoundMeta } from "./types/commit-meta.js";
|
||||
export type { CIAgentMetadata, ParsedCIAgentCommit, CommitType, CommitScope, CommitDecision, CommitEscalation, CommitRequirements, CommitCompoundMeta } from "./types/commit-meta.js";
|
||||
export type { ProjectState, BranchInfo } from "./core/git-context.js";
|
||||
export type { PhaseBranchInfo, MilestoneBranchInfo, BranchCreateResult, BranchMergeResult } from "./core/git-branch.js";
|
||||
export type { ProjectMd, RoadmapMd, RequirementsMd, ArchitectureMd } from "./core/ci-files.js";
|
||||
export type { ProjectMd, RoadmapMd, RequirementsMd, ArchitectureMd } from "./core/ciagent-files.js";
|
||||
export type { GiteaReleaseConfig, GiteaRelease } from "./core/gitea.js";
|
||||
export type { IntelligenceBackend, BackendRequest, BackendResult, BackendConfigSection, BackendUnavailableError, Artifact, TokenUsage } from "./backends/types.js";
|
||||
export type { ToolDefinition, ToolCall, ToolResult } from "./backends/tool-registry.js";
|
||||
@@ -51,7 +51,7 @@ export interface CommitCompoundMeta {
|
||||
solution: string;
|
||||
}
|
||||
|
||||
export interface CiMetadata {
|
||||
export interface CIAgentMetadata {
|
||||
phase: number;
|
||||
milestone: string;
|
||||
project?: string;
|
||||
@@ -65,12 +65,12 @@ export interface CiMetadata {
|
||||
compound?: CommitCompoundMeta;
|
||||
}
|
||||
|
||||
export interface ParsedCiCommit {
|
||||
export interface ParsedCIAgentCommit {
|
||||
hash: string;
|
||||
type: CommitType;
|
||||
scope: string;
|
||||
subject: string;
|
||||
ci: CiMetadata | null;
|
||||
ci: CIAgentMetadata | null;
|
||||
body: string;
|
||||
}
|
||||
|
||||
|
||||
+32
-32
@@ -1,33 +1,33 @@
|
||||
import { CIConfig, DEFAULT_CI_CONFIG, AutonomyLevel, ModelProfile, ProjectEntry } from "../types/config.js";
|
||||
import { CIAgentConfig, DEFAULT_CIAGENT_CONFIG, AutonomyLevel, ModelProfile, ProjectEntry } from "../types/config.js";
|
||||
|
||||
describe("CIConfig", () => {
|
||||
it("DEFAULT_CI_CONFIG has all required fields", () => {
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.level).toBe("full");
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.clarify_budget).toBe(10);
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.decision_confidence_threshold).toBe(0.6);
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.max_revision_iterations).toBe(3);
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.max_verification_retries).toBe(2);
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.escalation_timeout_ms).toBe(300000);
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.escalation_hooks).toContain("deploy");
|
||||
expect(DEFAULT_CI_CONFIG.model_profile).toBe("quality");
|
||||
expect(DEFAULT_CI_CONFIG.parallelization.enabled).toBe(true);
|
||||
expect(DEFAULT_CI_CONFIG.verification.automated_only).toBe(true);
|
||||
expect(DEFAULT_CI_CONFIG.security.auto_accept_low_severity).toBe(true);
|
||||
expect(DEFAULT_CI_CONFIG.git.auto_commit).toBe(true);
|
||||
expect(DEFAULT_CI_CONFIG.git.auto_push).toBe(false);
|
||||
describe("CIAgentConfig", () => {
|
||||
it("DEFAULT_CIAGENT_CONFIG has all required fields", () => {
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.level).toBe("full");
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.clarify_budget).toBe(10);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.decision_confidence_threshold).toBe(0.6);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.max_revision_iterations).toBe(3);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.max_verification_retries).toBe(2);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.escalation_timeout_ms).toBe(300000);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.escalation_hooks).toContain("deploy");
|
||||
expect(DEFAULT_CIAGENT_CONFIG.model_profile).toBe("quality");
|
||||
expect(DEFAULT_CIAGENT_CONFIG.parallelization.enabled).toBe(true);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.verification.automated_only).toBe(true);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.security.auto_accept_low_severity).toBe(true);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.git.auto_commit).toBe(true);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.git.auto_push).toBe(false);
|
||||
});
|
||||
|
||||
it("DEFAULT_CI_CONFIG has multi-project fields", () => {
|
||||
expect(DEFAULT_CI_CONFIG.projects).toEqual([]);
|
||||
expect(DEFAULT_CI_CONFIG.active_project).toBe("");
|
||||
it("DEFAULT_CIAGENT_CONFIG has multi-project fields", () => {
|
||||
expect(DEFAULT_CIAGENT_CONFIG.projects).toEqual([]);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.active_project).toBe("");
|
||||
});
|
||||
|
||||
it("AutonomyLevel accepts all valid levels", () => {
|
||||
const levels: AutonomyLevel[] = ["full", "supervised", "guided"];
|
||||
for (const level of levels) {
|
||||
const config: CIConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
autonomy: { ...DEFAULT_CI_CONFIG.autonomy, level },
|
||||
const config: CIAgentConfig = {
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
autonomy: { ...DEFAULT_CIAGENT_CONFIG.autonomy, level },
|
||||
};
|
||||
expect(config.autonomy.level).toBe(level);
|
||||
}
|
||||
@@ -36,8 +36,8 @@ describe("CIConfig", () => {
|
||||
it("ModelProfile accepts all valid profiles", () => {
|
||||
const profiles: ModelProfile[] = ["quality", "speed", "balanced"];
|
||||
for (const profile of profiles) {
|
||||
const config: CIConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
const config: CIAgentConfig = {
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
model_profile: profile,
|
||||
};
|
||||
expect(config.model_profile).toBe(profile);
|
||||
@@ -45,7 +45,7 @@ describe("CIConfig", () => {
|
||||
});
|
||||
|
||||
it("escalation_hooks defaults include expected items", () => {
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.escalation_hooks).toEqual([
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.escalation_hooks).toEqual([
|
||||
"deploy",
|
||||
"delete_data",
|
||||
"merge_to_main",
|
||||
@@ -66,10 +66,10 @@ describe("CIConfig", () => {
|
||||
});
|
||||
});
|
||||
|
||||
describe("CIConfig with projects", () => {
|
||||
describe("CIAgentConfig with projects", () => {
|
||||
it("supports multiple projects", () => {
|
||||
const config: CIConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
const config: CIAgentConfig = {
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
projects: [
|
||||
{ slug: "task-api", name: "Task API", default: true },
|
||||
{ slug: "auth-svc", name: "Auth Service" },
|
||||
@@ -82,8 +82,8 @@ describe("CIConfig", () => {
|
||||
});
|
||||
|
||||
it("supports single project", () => {
|
||||
const config: CIConfig = {
|
||||
...DEFAULT_CI_CONFIG,
|
||||
const config: CIAgentConfig = {
|
||||
...DEFAULT_CIAGENT_CONFIG,
|
||||
projects: [{ slug: "my-app", name: "My App", default: true }],
|
||||
active_project: "my-app",
|
||||
};
|
||||
@@ -92,8 +92,8 @@ describe("CIConfig", () => {
|
||||
});
|
||||
|
||||
it("defaults to empty projects array and empty active_project", () => {
|
||||
expect(DEFAULT_CI_CONFIG.projects).toEqual([]);
|
||||
expect(DEFAULT_CI_CONFIG.active_project).toBe("");
|
||||
expect(DEFAULT_CIAGENT_CONFIG.projects).toEqual([]);
|
||||
expect(DEFAULT_CIAGENT_CONFIG.active_project).toBe("");
|
||||
});
|
||||
});
|
||||
});
|
||||
+18
-3
@@ -28,7 +28,8 @@ export type AgentName =
|
||||
| "plan-checker"
|
||||
| "project-researcher"
|
||||
| "research-synthesizer"
|
||||
| "solution-writer";
|
||||
| "solution-writer"
|
||||
| "tester";
|
||||
|
||||
export interface AutonomyConfig {
|
||||
level: AutonomyLevel;
|
||||
@@ -65,13 +66,20 @@ export interface GitConfig {
|
||||
auto_push: boolean;
|
||||
}
|
||||
|
||||
export interface GiteaConfig {
|
||||
base_url: string;
|
||||
api_token_env: string;
|
||||
owner: string;
|
||||
repo: string;
|
||||
}
|
||||
|
||||
export interface ProjectEntry {
|
||||
slug: string;
|
||||
name: string;
|
||||
default?: boolean;
|
||||
}
|
||||
|
||||
export interface CIConfig {
|
||||
export interface CIAgentConfig {
|
||||
projects: ProjectEntry[];
|
||||
active_project: string;
|
||||
autonomy: AutonomyConfig;
|
||||
@@ -81,9 +89,10 @@ export interface CIConfig {
|
||||
security: SecurityConfig;
|
||||
git: GitConfig;
|
||||
backend: BackendConfigSection;
|
||||
gitea?: GiteaConfig;
|
||||
}
|
||||
|
||||
export const DEFAULT_CI_CONFIG: CIConfig = {
|
||||
export const DEFAULT_CIAGENT_CONFIG: CIAgentConfig = {
|
||||
projects: [],
|
||||
active_project: "",
|
||||
autonomy: {
|
||||
@@ -135,4 +144,10 @@ export const DEFAULT_CI_CONFIG: CIConfig = {
|
||||
},
|
||||
},
|
||||
},
|
||||
gitea: {
|
||||
base_url: "https://git.cloudinit.dev",
|
||||
api_token_env: "GITEA_TOKEN",
|
||||
owner: "",
|
||||
repo: "",
|
||||
},
|
||||
};
|
||||
@@ -3,11 +3,11 @@ import { confidenceToLevel, shouldEscalate } from "../types/decisions.js";
|
||||
import { ESCALATION_TYPES } from "../types/escalation.js";
|
||||
import { parseSpecification } from "../types/specification.js";
|
||||
import { createClarifyQuestion } from "../types/clarify.js";
|
||||
import { DEFAULT_CI_CONFIG } from "../types/config.js";
|
||||
import { DEFAULT_CIAGENT_CONFIG } from "../types/config.js";
|
||||
|
||||
describe("Type exports", () => {
|
||||
it("pipeline types are importable and functional", () => {
|
||||
expect(STAGE_ORDER).toHaveLength(7);
|
||||
expect(STAGE_ORDER).toHaveLength(8);
|
||||
expect(getNextStage("specify")).toBe("clarify");
|
||||
const state = createInitialPipelineState("/tmp/test");
|
||||
expect(state.current_stage).toBe("specify");
|
||||
@@ -40,6 +40,6 @@ describe("Type exports", () => {
|
||||
});
|
||||
|
||||
it("config defaults are importable", () => {
|
||||
expect(DEFAULT_CI_CONFIG.autonomy.level).toBe("full");
|
||||
expect(DEFAULT_CIAGENT_CONFIG.autonomy.level).toBe("full");
|
||||
});
|
||||
});
|
||||
@@ -8,13 +8,14 @@ import {
|
||||
} from "../types/pipeline.js";
|
||||
|
||||
describe("STAGE_ORDER", () => {
|
||||
it("has 7 stages in correct order", () => {
|
||||
it("has 8 stages in correct order", () => {
|
||||
expect(STAGE_ORDER).toEqual([
|
||||
"specify",
|
||||
"clarify",
|
||||
"research",
|
||||
"plan",
|
||||
"execute",
|
||||
"test",
|
||||
"verify",
|
||||
"complete",
|
||||
]);
|
||||
@@ -27,7 +28,8 @@ describe("getNextStage", () => {
|
||||
expect(getNextStage("clarify")).toBe("research");
|
||||
expect(getNextStage("research")).toBe("plan");
|
||||
expect(getNextStage("plan")).toBe("execute");
|
||||
expect(getNextStage("execute")).toBe("verify");
|
||||
expect(getNextStage("execute")).toBe("test");
|
||||
expect(getNextStage("test")).toBe("verify");
|
||||
expect(getNextStage("verify")).toBe("complete");
|
||||
});
|
||||
|
||||
@@ -51,6 +53,7 @@ describe("createInitialPipelineState", () => {
|
||||
expect(state.research_completed).toBe(false);
|
||||
expect(state.plan_completed).toBe(false);
|
||||
expect(state.execute_completed).toBe(false);
|
||||
expect(state.test_completed).toBe(false);
|
||||
expect(state.verify_completed).toBe(false);
|
||||
expect(state.errors).toHaveLength(0);
|
||||
expect(state.started_at).toBeTruthy();
|
||||
|
||||
@@ -6,6 +6,7 @@ export type PipelineStage =
|
||||
| "research"
|
||||
| "plan"
|
||||
| "execute"
|
||||
| "test"
|
||||
| "verify"
|
||||
| "complete";
|
||||
|
||||
@@ -19,6 +20,7 @@ export interface PipelineState {
|
||||
research_completed: boolean;
|
||||
plan_completed: boolean;
|
||||
execute_completed: boolean;
|
||||
test_completed: boolean;
|
||||
verify_completed: boolean;
|
||||
errors: PipelineError[];
|
||||
started_at: string;
|
||||
@@ -61,6 +63,7 @@ export const STAGE_ORDER: PipelineStage[] = [
|
||||
"research",
|
||||
"plan",
|
||||
"execute",
|
||||
"test",
|
||||
"verify",
|
||||
"complete",
|
||||
];
|
||||
@@ -84,6 +87,7 @@ export function createInitialPipelineState(
|
||||
research_completed: false,
|
||||
plan_completed: false,
|
||||
execute_completed: false,
|
||||
test_completed: false,
|
||||
verify_completed: false,
|
||||
errors: [],
|
||||
started_at: new Date().toISOString(),
|
||||
|
||||
@@ -7,7 +7,7 @@ describe("file utilities", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-file-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-file-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
@@ -115,8 +115,8 @@ describe("file utilities", () => {
|
||||
expect(getProjectRoot(path.join(tempDir, "subdir"))).toBe(tempDir);
|
||||
});
|
||||
|
||||
it("finds project root with .ci directory", () => {
|
||||
fs.mkdirSync(path.join(tempDir, ".ci"));
|
||||
it("finds project root with .ciagent directory", () => {
|
||||
fs.mkdirSync(path.join(tempDir, ".ciagent"));
|
||||
expect(getProjectRoot(path.join(tempDir, "nested", "dir"))).toBe(tempDir);
|
||||
});
|
||||
|
||||
|
||||
+1
-1
@@ -47,7 +47,7 @@ export function copyFile(src: string, dest: string): void {
|
||||
export function getProjectRoot(startPath?: string): string {
|
||||
let current = startPath || process.cwd();
|
||||
while (current !== path.dirname(current)) {
|
||||
if (fs.existsSync(path.join(current, ".ci"))) return current;
|
||||
if (fs.existsSync(path.join(current, ".ciagent"))) return current;
|
||||
if (fs.existsSync(path.join(current, ".git"))) return current;
|
||||
if (fs.existsSync(path.join(current, "package.json"))) return current;
|
||||
current = path.dirname(current);
|
||||
|
||||
@@ -7,7 +7,7 @@ describe("BehavioralVerification", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-behavioral-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-behavioral-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
@@ -50,7 +50,7 @@ describe("BehavioralVerification", () => {
|
||||
});
|
||||
|
||||
it("passes with REQUIREMENTS.md", async () => {
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "REQUIREMENTS.md"), "# Requirements\n\n| REQ-ID | Requirement | Priority | Phase | Status |\n|--------|-------------|----------|-------|--------|\n| REQ-01 | Must have auth | P0 | 1 | pending |\n");
|
||||
|
||||
@@ -62,7 +62,7 @@ describe("BehavioralVerification", () => {
|
||||
});
|
||||
|
||||
it("skips when no REQUIREMENTS.md or PROJECT.md", async () => {
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
|
||||
const verifier = new BehavioralVerification();
|
||||
@@ -73,7 +73,7 @@ describe("BehavioralVerification", () => {
|
||||
});
|
||||
|
||||
it("passes with PROJECT.md when no REQUIREMENTS.md", async () => {
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "PROJECT.md"), "# Test\n\n## What This Is\nBuild it\n\n## Requirements\n\n### Active\n\n- [ ] Must have auth\n- [ ] Shall support CRUD\n");
|
||||
|
||||
|
||||
@@ -108,8 +108,8 @@ export class BehavioralVerification extends VerificationLayer {
|
||||
}
|
||||
|
||||
private checkSpecificationRequirements(projectPath: string): VerificationCheck {
|
||||
const reqPath = path.join(projectPath, ".ci", "REQUIREMENTS.md");
|
||||
const projectPath_md = path.join(projectPath, ".ci", "PROJECT.md");
|
||||
const reqPath = path.join(projectPath, ".ciagent", "REQUIREMENTS.md");
|
||||
const projectPath_md = path.join(projectPath, ".ciagent", "PROJECT.md");
|
||||
|
||||
const specPath = reqPath;
|
||||
if (!fs.existsSync(specPath)) {
|
||||
@@ -189,7 +189,7 @@ export class BehavioralVerification extends VerificationLayer {
|
||||
private checkPlanMustHaves(projectPath: string, phase: number): VerificationCheck {
|
||||
const roadmapPath = path.join(
|
||||
projectPath,
|
||||
".ci",
|
||||
".ciagent",
|
||||
"ROADMAP.md"
|
||||
);
|
||||
|
||||
@@ -252,7 +252,7 @@ export class BehavioralVerification extends VerificationLayer {
|
||||
ciBlockRegex.lastIndex = 0;
|
||||
}
|
||||
|
||||
const reqPath = path.join(projectPath, ".ci", "REQUIREMENTS.md");
|
||||
const reqPath = path.join(projectPath, ".ciagent", "REQUIREMENTS.md");
|
||||
if (!fs.existsSync(reqPath)) {
|
||||
return this.check(
|
||||
"Requirement test coverage via git log",
|
||||
|
||||
@@ -7,8 +7,8 @@ describe("VerificationPipeline", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-pipeline-test-"));
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-pipeline-test-"));
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), JSON.stringify({ autonomy: { level: "full" } }));
|
||||
fs.writeFileSync(path.join(ciDir, "specification.md"), "# Test\n## Objective\nBuild it\n\n## Requirements\n- Feature A\n");
|
||||
|
||||
@@ -7,7 +7,7 @@ describe("QualityVerification", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-quality-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-quality-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
|
||||
@@ -7,7 +7,7 @@ describe("SecurityVerification", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-security-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-security-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
|
||||
@@ -7,28 +7,28 @@ describe("StructuralVerification", () => {
|
||||
let tempDir: string;
|
||||
|
||||
beforeEach(() => {
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ci-structural-test-"));
|
||||
tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "ciagent-structural-test-"));
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
fs.rmSync(tempDir, { recursive: true, force: true });
|
||||
});
|
||||
|
||||
function setupProjectStructure(hasCIDir = true, hasRoadmap = true, hasCIConfig = true, hasSpec = true) {
|
||||
function setupProjectStructure(hasCIDir = true, hasRoadmap = true, hasCIAgentConfig = true, hasSpec = true) {
|
||||
if (hasCIDir) {
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
if (hasRoadmap) {
|
||||
fs.writeFileSync(path.join(ciDir, "ROADMAP.md"), "# Roadmap\n\n## Phases\n\n### Phase 1: Init\n**Goal**: Set up project\n**Status**: not_started\n");
|
||||
}
|
||||
}
|
||||
if (hasCIConfig) {
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
if (hasCIAgentConfig) {
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), JSON.stringify({ autonomy: { level: "full" } }, null, 2));
|
||||
}
|
||||
if (hasSpec) {
|
||||
const specDir = path.join(tempDir, ".ci");
|
||||
const specDir = path.join(tempDir, ".ciagent");
|
||||
if (!fs.existsSync(specDir)) fs.mkdirSync(specDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(specDir, "specification.md"), "# Project\n## Objective\nBuild a REST API for task management\n\n## Requirements\n- User auth\n- CRUD\n");
|
||||
}
|
||||
@@ -43,13 +43,13 @@ describe("StructuralVerification", () => {
|
||||
expect(result.name).toBe("Structural");
|
||||
expect(result.checks.length).toBeGreaterThan(0);
|
||||
|
||||
const phaseDirCheck = result.checks.find((c) => c.name === ".ci directory exists");
|
||||
const phaseDirCheck = result.checks.find((c) => c.name === ".ciagent directory exists");
|
||||
expect(phaseDirCheck?.status).toBe("pass");
|
||||
|
||||
const planCheck = result.checks.find((c) => c.name === "ROADMAP.md exists");
|
||||
expect(planCheck?.status).toBe("pass");
|
||||
|
||||
const configCheck = result.checks.find((c) => c.name === "CI config valid");
|
||||
const configCheck = result.checks.find((c) => c.name === "CIAgent config valid");
|
||||
expect(configCheck?.status).toBe("pass");
|
||||
|
||||
const specCheck = result.checks.find((c) => c.name === "Specification exists");
|
||||
@@ -61,7 +61,7 @@ describe("StructuralVerification", () => {
|
||||
const verifier = new StructuralVerification();
|
||||
const result = await verifier.verify(tempDir, 1);
|
||||
|
||||
const phaseDirCheck = result.checks.find((c) => c.name === ".ci directory exists");
|
||||
const phaseDirCheck = result.checks.find((c) => c.name === ".ciagent directory exists");
|
||||
expect(phaseDirCheck?.status).toBe("fail");
|
||||
});
|
||||
|
||||
@@ -75,15 +75,15 @@ describe("StructuralVerification", () => {
|
||||
});
|
||||
|
||||
it("fails when CI config has invalid JSON", async () => {
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), "not valid json{{{");
|
||||
fs.mkdirSync(path.join(tempDir, ".ci"), { recursive: true });
|
||||
fs.mkdirSync(path.join(tempDir, ".ciagent"), { recursive: true });
|
||||
|
||||
const verifier = new StructuralVerification();
|
||||
const result = await verifier.verify(tempDir, 1);
|
||||
|
||||
const configCheck = result.checks.find((c) => c.name === "CI config valid");
|
||||
const configCheck = result.checks.find((c) => c.name === "CIAgent config valid");
|
||||
expect(configCheck?.status).toBe("fail");
|
||||
});
|
||||
|
||||
@@ -91,7 +91,7 @@ describe("StructuralVerification", () => {
|
||||
const srcDir = path.join(tempDir, "src");
|
||||
fs.mkdirSync(srcDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(srcDir, "app.ts"), "export function main() { /* TODO: implement */ }");
|
||||
const ciDir = path.join(tempDir, ".ci");
|
||||
const ciDir = path.join(tempDir, ".ciagent");
|
||||
fs.mkdirSync(ciDir, { recursive: true });
|
||||
fs.writeFileSync(path.join(ciDir, "config.json"), "{}");
|
||||
fs.writeFileSync(path.join(ciDir, "specification.md"), "# Test\n## Objective\nBuild it");
|
||||
|
||||
@@ -23,7 +23,7 @@ export class StructuralVerification extends VerificationLayer {
|
||||
|
||||
checks.push(this.checkPhaseDir(projectPath, phase));
|
||||
checks.push(this.checkPlanExists(projectPath, phase));
|
||||
checks.push(this.checkCIConfig(projectPath));
|
||||
checks.push(this.checkCIAgentConfig(projectPath));
|
||||
checks.push(this.checkSpecification(projectPath));
|
||||
checks.push(this.checkNoStubs(projectPath));
|
||||
checks.push(this.checkImportsWired(projectPath));
|
||||
@@ -41,47 +41,47 @@ export class StructuralVerification extends VerificationLayer {
|
||||
}
|
||||
|
||||
private checkPhaseDir(projectPath: string, phase: number) {
|
||||
const ciDir = path.join(projectPath, ".ci");
|
||||
const ciDir = path.join(projectPath, ".ciagent");
|
||||
const exists = fs.existsSync(ciDir);
|
||||
return this.check(
|
||||
".ci directory exists",
|
||||
".ciagent directory exists",
|
||||
exists ? "pass" : "fail",
|
||||
exists ? ".ci directory found" : ".ci directory not found",
|
||||
exists ? ".ciagent directory found" : ".ciagent directory not found",
|
||||
ciDir
|
||||
);
|
||||
}
|
||||
|
||||
private checkPlanExists(projectPath: string, phase: number) {
|
||||
const roadmapPath = path.join(projectPath, ".ci", "ROADMAP.md");
|
||||
const roadmapPath = path.join(projectPath, ".ciagent", "ROADMAP.md");
|
||||
const exists = fs.existsSync(roadmapPath);
|
||||
return this.check(
|
||||
"ROADMAP.md exists",
|
||||
exists ? "pass" : "warning",
|
||||
exists ? "ROADMAP.md found" : "ROADMAP.md not found (run 'ci init' first)",
|
||||
exists ? "ROADMAP.md found" : "ROADMAP.md not found (run 'ciagent init' first)",
|
||||
roadmapPath
|
||||
);
|
||||
}
|
||||
|
||||
private checkCIConfig(projectPath: string) {
|
||||
const configPath = path.join(projectPath, ".ci", "config.json");
|
||||
private checkCIAgentConfig(projectPath: string) {
|
||||
const configPath = path.join(projectPath, ".ciagent", "config.json");
|
||||
const exists = fs.existsSync(configPath);
|
||||
if (!exists) {
|
||||
return this.check("CI config exists", "fail", ".ci/config.json not found", configPath);
|
||||
return this.check("CIAgent config exists", "fail", ".ciagent/config.json not found", configPath);
|
||||
}
|
||||
try {
|
||||
const content = fs.readFileSync(configPath, "utf-8");
|
||||
JSON.parse(content);
|
||||
return this.check("CI config valid", "pass", ".ci/config.json is valid JSON");
|
||||
return this.check("CIAgent config valid", "pass", ".ciagent/config.json is valid JSON");
|
||||
} catch (e) {
|
||||
return this.check("CI config valid", "fail", `.ci/config.json has invalid JSON: ${(e as Error).message}`);
|
||||
return this.check("CIAgent config valid", "fail", `.ciagent/config.json has invalid JSON: ${(e as Error).message}`);
|
||||
}
|
||||
}
|
||||
|
||||
private checkSpecification(projectPath: string) {
|
||||
const specPath = path.join(projectPath, ".ci", "specification.md");
|
||||
const specPath = path.join(projectPath, ".ciagent", "specification.md");
|
||||
const exists = fs.existsSync(specPath);
|
||||
if (!exists) {
|
||||
return this.check("Specification exists", "warning", ".ci/specification.md not found — specification may not be loaded yet");
|
||||
return this.check("Specification exists", "warning", ".ciagent/specification.md not found — specification may not be loaded yet");
|
||||
}
|
||||
const content = fs.readFileSync(specPath, "utf-8");
|
||||
if (content.trim().length < 10) {
|
||||
|
||||
+1
-1
@@ -1 +1 @@
|
||||
export const VERSION = "0.5.0";
|
||||
export const VERSION = "0.7.0";
|
||||
@@ -4,14 +4,13 @@
|
||||
|
||||
## Decision Log
|
||||
|
||||
Decisions are automatically logged to `.ciagent/audit/` with:
|
||||
Decisions are automatically logged to git commits via `---ci---` YAML blocks with:
|
||||
- Timestamp
|
||||
- Decision ID
|
||||
- What was decided
|
||||
- Why (reasoning chain)
|
||||
- Confidence level
|
||||
- What alternatives were considered
|
||||
- What the human would have been asked in Learnship mode
|
||||
|
||||
## Reviewing Decisions
|
||||
|
||||
|
||||
Reference in New Issue
Block a user