mirror of
https://github.com/Fission-AI/OpenSpec.git
synced 2026-09-14 20:16:53 +08:00
af273b8e0b
* proposal: add artifact graph core query system
Add OpenSpec change proposal for Slice 1 of the artifact POC - the core
"What's Ready?" query system. This implements:
- ArtifactGraph class for DAG-based dependency modeling
- Filesystem-based state detection (file existence = completion)
- Topological sort for build order calculation
- Ready/blocked artifact queries
This is a parallel module that will coexist with the current system.
* docs: specify Zod for schema validation in artifact graph proposal
- Add decision section for Zod schema validation in design.md
- Update data structures to show Zod schemas with z.infer<> types
- Update tasks to specify Zod usage for type definitions and parsing
* docs: add 2-level schema resolution and built-in schemas
- Add decision for global → built-in schema resolution pattern
- Add resolver.ts for schema lookup logic
- Add built-in schemas directory (spec-driven.yaml, tdd.yaml)
- Add schema resolution tests
- Follows ESLint/Prettier/Git patterns (defaults baked in package)
* experiment: add vertical slice version of artifact graph change
Creates add-artifact-graph-core-v2 with requirements organized as
vertical slices - each requirement file contains its spec, design
decisions, and tasks bundled together for comparison.
* feat(core): add getGlobalDataDir for XDG-compliant data directory
Add getGlobalDataDir() function following XDG Base Directory Specification
for storing user data like schema overrides:
- XDG_DATA_HOME takes precedence on all platforms
- Unix/macOS fallback: ~/.local/share/openspec/
- Windows fallback: %LOCALAPPDATA%/openspec/
* feat(artifact-graph): add core dependency graph module
Implement Slice 1 ("What's Ready?") of the artifact graph system:
- types.ts: Zod schemas for artifact definitions with derived TypeScript types
- schema.ts: YAML parsing with validation for duplicates, invalid refs, cycles
- graph.ts: ArtifactGraph class with Kahn's algorithm for topological sort
- state.ts: Filesystem-based completion detection with glob pattern support
- resolver.ts: Two-level schema resolution (global override → built-in)
- builtin-schemas.ts: spec-driven and tdd workflow definitions
Key design decisions:
- Filesystem as database (stateless, git-friendly)
- Cycle errors show full path (e.g., "A → B → C → A")
- Deterministic ordering via sorted queues
* test(artifact-graph): add comprehensive test suite
52 tests covering all artifact-graph functionality:
- schema.test.ts: Parsing, validation errors, cycle detection
- graph.test.ts: Build order, ready artifacts, blocked queries
- state.test.ts: File existence, glob patterns, missing directories
- resolver.test.ts: Schema resolution with global overrides
* docs(openspec): archive add-artifact-graph-core change
Archive completed change proposal and create artifact-graph spec with
6 requirements covering schema loading, build order, state detection,
ready queries, completion checks, and blocked queries.
* chore: remove experimental artifact-graph-core-v2 folder
Clean up experimental vertical slice proposal that is no longer needed.
* feat(artifact-graph): validate global schema overrides
Global schema overrides are now validated through the same pipeline as
built-in schemas, catching invalid schemas, cyclic dependencies, and
invalid requires references at load time. Added SchemaLoadError for
better error context with file paths.
* test(artifact-graph): add workflow integration tests
Add end-to-end integration tests that exercise the full artifact-graph
pipeline: resolveSchema → ArtifactGraph → detectCompleted → queries.
Tests cover:
- Complete spec-driven and tdd workflow progressions
- Out-of-order file creation handling
- Glob pattern matching with multiple files
- Build order consistency
- Edge cases (empty/missing directories, non-matching files)
* refactor(artifact-graph): adopt zod v4 error message format
Update custom error messages from string format to zod v4 object format
using `{ error: 'message' }` convention.
* fix(test): prevent hanging vitest threads after test runs
- Add teardownTimeout (3s) to vitest config for forced cleanup
- Add global teardown function to vitest.setup.ts
- Call child.unref() to prevent child processes from blocking event loop
- Explicitly destroy stdio streams on process close/error
151 lines
3.8 KiB
TypeScript
151 lines
3.8 KiB
TypeScript
import { spawn } from 'child_process';
|
|
import { existsSync } from 'fs';
|
|
import path from 'path';
|
|
import { fileURLToPath } from 'url';
|
|
|
|
const __filename = fileURLToPath(import.meta.url);
|
|
const __dirname = path.dirname(__filename);
|
|
|
|
const projectRoot = path.resolve(__dirname, '..', '..');
|
|
const cliEntry = path.join(projectRoot, 'dist', 'cli', 'index.js');
|
|
|
|
let buildPromise: Promise<void> | undefined;
|
|
|
|
interface RunCommandOptions {
|
|
cwd?: string;
|
|
env?: NodeJS.ProcessEnv;
|
|
}
|
|
|
|
interface RunCLIOptions {
|
|
cwd?: string;
|
|
env?: NodeJS.ProcessEnv;
|
|
input?: string;
|
|
timeoutMs?: number;
|
|
}
|
|
|
|
export interface RunCLIResult {
|
|
exitCode: number | null;
|
|
signal: NodeJS.Signals | null;
|
|
stdout: string;
|
|
stderr: string;
|
|
timedOut: boolean;
|
|
command: string;
|
|
}
|
|
|
|
function runCommand(command: string, args: string[], options: RunCommandOptions = {}) {
|
|
return new Promise<void>((resolve, reject) => {
|
|
const child = spawn(command, args, {
|
|
cwd: options.cwd ?? projectRoot,
|
|
env: { ...process.env, ...options.env },
|
|
stdio: 'inherit',
|
|
shell: process.platform === 'win32',
|
|
});
|
|
|
|
child.on('error', (error) => reject(error));
|
|
child.on('close', (code, signal) => {
|
|
if (code === 0) {
|
|
resolve();
|
|
} else {
|
|
const reason = signal ? `signal ${signal}` : `exit code ${code}`;
|
|
reject(new Error(`Command failed (${reason}): ${command} ${args.join(' ')}`));
|
|
}
|
|
});
|
|
});
|
|
}
|
|
|
|
export async function ensureCliBuilt() {
|
|
if (existsSync(cliEntry)) {
|
|
return;
|
|
}
|
|
|
|
if (!buildPromise) {
|
|
buildPromise = runCommand('pnpm', ['run', 'build']).catch((error) => {
|
|
buildPromise = undefined;
|
|
throw error;
|
|
});
|
|
}
|
|
|
|
await buildPromise;
|
|
|
|
if (!existsSync(cliEntry)) {
|
|
throw new Error('CLI entry point missing after build. Expected dist/cli/index.js');
|
|
}
|
|
}
|
|
|
|
export async function runCLI(args: string[] = [], options: RunCLIOptions = {}): Promise<RunCLIResult> {
|
|
await ensureCliBuilt();
|
|
|
|
const finalArgs = Array.isArray(args) ? args : [args];
|
|
const invocation = [cliEntry, ...finalArgs].join(' ');
|
|
|
|
return new Promise<RunCLIResult>((resolve, reject) => {
|
|
const child = spawn(process.execPath, [cliEntry, ...finalArgs], {
|
|
cwd: options.cwd ?? projectRoot,
|
|
env: {
|
|
...process.env,
|
|
OPEN_SPEC_INTERACTIVE: '0',
|
|
...options.env,
|
|
},
|
|
stdio: ['pipe', 'pipe', 'pipe'],
|
|
windowsHide: true,
|
|
});
|
|
|
|
// Prevent child process from keeping the event loop alive
|
|
child.unref();
|
|
|
|
let stdout = '';
|
|
let stderr = '';
|
|
let timedOut = false;
|
|
|
|
const timeout = options.timeoutMs
|
|
? setTimeout(() => {
|
|
timedOut = true;
|
|
child.kill('SIGKILL');
|
|
}, options.timeoutMs)
|
|
: undefined;
|
|
|
|
child.stdout?.setEncoding('utf-8');
|
|
child.stdout?.on('data', (chunk) => {
|
|
stdout += chunk;
|
|
});
|
|
|
|
child.stderr?.setEncoding('utf-8');
|
|
child.stderr?.on('data', (chunk) => {
|
|
stderr += chunk;
|
|
});
|
|
|
|
child.on('error', (error) => {
|
|
if (timeout) clearTimeout(timeout);
|
|
// Explicitly destroy streams to prevent hanging handles
|
|
child.stdout?.destroy();
|
|
child.stderr?.destroy();
|
|
child.stdin?.destroy();
|
|
reject(error);
|
|
});
|
|
|
|
child.on('close', (code, signal) => {
|
|
if (timeout) clearTimeout(timeout);
|
|
// Explicitly destroy streams to prevent hanging handles
|
|
child.stdout?.destroy();
|
|
child.stderr?.destroy();
|
|
child.stdin?.destroy();
|
|
resolve({
|
|
exitCode: code,
|
|
signal,
|
|
stdout,
|
|
stderr,
|
|
timedOut,
|
|
command: `node ${invocation}`,
|
|
});
|
|
});
|
|
|
|
if (options.input && child.stdin) {
|
|
child.stdin.end(options.input);
|
|
} else if (child.stdin) {
|
|
child.stdin.end();
|
|
}
|
|
});
|
|
}
|
|
|
|
export const cliProjectRoot = projectRoot;
|