dreamteamer 0.6.0 → 0.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "dreamteamer",
-	"version": "0.6.0",
+	"version": "0.6.1",
 	"description": "A workspace compiler for coding agents — schema-validated records as plain files over git, compiled into every harness",
 	"license": "Apache-2.0",
 	"author": "Gilad Khen <giladkhen@gmail.com>",

package/src/cli.js

@@ -12,6 +12,12 @@ import { deriveEvents } from './events.js';
 import { commitPending } from './commit.js';
 import { Store } from './store.js';
 
+// git calls whose failure we CATCH must not print git's own error: execFileSync forwards the
+// child's stderr to ours unless told otherwise, so a handled "not a git repository" still
+// reached the user's terminal. stdout stays piped because we read it.
+const QUIET = ['ignore', 'pipe', 'ignore'];
+
+
 const USAGE = `usage: dreamteamer <command> | dreamteamer <collection> <verb> …
 
 commands:
@@ -230,7 +236,7 @@ export function run(argv) {
 }
 
 function tryGit(cwd, args) {
-	try { return execFileSync('git', args, { cwd }).toString().trim() || null; } catch { return null; }
+	try { return execFileSync('git', args, { cwd, stdio: QUIET }).toString().trim() || null; } catch { return null; }
 }
 
 function watchAndRecompile(ws) {

package/src/commit.js

@@ -6,6 +6,12 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { pathToRecord } from './events.js';
 
+// git calls whose failure we CATCH must not print git's own error: execFileSync forwards the
+// child's stderr to ours unless told otherwise, so a handled "not a git repository" still
+// reached the user's terminal. stdout stays piped because we read it.
+const QUIET = ['ignore', 'pipe', 'ignore'];
+
+
 const VERB = { A: 'add', M: 'set', D: 'rm', R: 'rename', '?': 'add' };
 
 /** Record directories to watch, grouped by owning repo. System-stored collections are excluded:
@@ -31,7 +37,7 @@ function scopeByRepo(descriptors, only) {
  *  is NOT a refusal condition — committing dirty records is this verb's entire job. */
 function inProgress(cwd) {
 	let gitDir;
-	try { gitDir = execFileSync('git', ['rev-parse', '--absolute-git-dir'], { cwd }).toString().trim(); }
+	try { gitDir = execFileSync('git', ['rev-parse', '--absolute-git-dir'], { cwd, stdio: QUIET }).toString().trim(); }
 	catch { return 'not a git repository'; }
 	for (const [marker, label] of [['MERGE_HEAD', 'merge'], ['rebase-merge', 'rebase'], ['rebase-apply', 'rebase'], ['CHERRY_PICK_HEAD', 'cherry-pick'], ['REVERT_HEAD', 'revert']]) {
 		if (fs.existsSync(path.join(gitDir, marker))) return `a ${label} is in progress`;
@@ -42,7 +48,7 @@ function inProgress(cwd) {
 /** A commit on a detached HEAD is reachable only by sha. Worth saying out loud before making one;
  *  not worth refusing over, since it is sometimes exactly what someone means to do. */
 function detached(cwd) {
-	try { return execFileSync('git', ['symbolic-ref', '--quiet', 'HEAD'], { cwd }).toString().trim() === ''; }
+	try { return execFileSync('git', ['symbolic-ref', '--quiet', 'HEAD'], { cwd, stdio: QUIET }).toString().trim() === ''; }
 	catch { return true; }
 }
 
@@ -57,7 +63,10 @@ function sample(root, repo, dirs, descriptors) {
 	// `-uall` is load-bearing: by default porcelain COLLAPSES an untracked directory to a single
 	// `?? data/notes/` entry, which maps to no record — so the first records of a brand-new
 	// collection would be invisible and dt commit would report success having committed nothing.
-	const out = execFileSync('git', ['status', '--porcelain', '-z', '-uall', '--', ...relDirs], { cwd }).toString();
+	// QUIET: the caller (cli `status`) catches a failure here so it can still print the rest of the
+	// report in a non-git folder — but git's own "fatal: not a git repository" was reaching the
+	// terminal anyway, which made a handled case look like a crash.
+	const out = execFileSync('git', ['status', '--porcelain', '-z', '-uall', '--', ...relDirs], { cwd, stdio: QUIET }).toString();
 	const chunks = out.split('\0').filter((c) => c.length > 0);
 	const rows = [];
 	for (let i = 0; i < chunks.length; i++) {

package/src/init.js

@@ -9,6 +9,12 @@ import { slugOrHash } from './template.js';
 import { discoverModules, KINDS } from './compile.js';
 import { Store } from './store.js';
 
+// git calls whose failure we CATCH must not print git's own error: execFileSync forwards the
+// child's stderr to ours unless told otherwise, so a handled "not a git repository" still
+// reached the user's terminal. stdout stays piped because we read it.
+const QUIET = ['ignore', 'pipe', 'ignore'];
+
+
 const SKELETON_KINDS = ['collections', 'skills', 'agents', 'commands', 'ui-views'];
 
 const GITIGNORE = `node_modules/
@@ -200,7 +206,7 @@ function buildClone(dest, name) {
 }
 
 function tryGit(cwd, args) {
-	try { return execFileSync('git', args, { cwd }).toString().trim() || null; } catch { return null; }
+	try { return execFileSync('git', args, { cwd, stdio: QUIET }).toString().trim() || null; } catch { return null; }
 }
 
 function appendMissing(file, block) {

package/src/server.js

@@ -16,6 +16,12 @@ import { commandsFor, recordResolver } from './record-commands.js';
 import { distinctValues } from './field-values.js';
 import { slugOrHash } from './template.js';
 
+// git calls whose failure we CATCH must not print git's own error: execFileSync forwards the
+// child's stderr to ours unless told otherwise, so a handled "not a git repository" still
+// reached the user's terminal. stdout stays piped because we read it.
+const QUIET = ['ignore', 'pipe', 'ignore'];
+
+
 export function startServer(ws, { port = 8080, host = '127.0.0.1' } = {}) {
 	const app = express();
 	app.use(express.json({ limit: '10mb' }));
@@ -42,7 +48,7 @@ export function startServer(ws, { port = 8080, host = '127.0.0.1' } = {}) {
 	// so `@me` filters in ui-views resolve to the seeded user record.
 	let operatorId = null;
 	try {
-		operatorId = slugOrHash(execFileSync('git', ['config', 'user.name'], { cwd: ws.root }).toString().trim());
+		operatorId = slugOrHash(execFileSync('git', ['config', 'user.name'], { cwd: ws.root, stdio: QUIET }).toString().trim());
 	} catch { /* no git identity — @me filters simply won't narrow */ }
 
 	api.get('/info', (req, res) => {

package/src/store.js

@@ -12,6 +12,12 @@ import { generateId } from './template.js';
 import { parseRecord, parseRecordText, patternRe, fmtAjvError, unknownFields, walk, EXT, assertSafeId } from './records.js';
 import { normalizeRecord } from './temporal.js';
 import { NO_RUNTIME, loadDescriptors, runtimeDir, sourceRoots as compiledSourceRoots } from './runtime.js';
+
+// git calls whose failure we CATCH must not print git's own error: execFileSync forwards the
+// child's stderr to ours unless told otherwise, so a handled "not a git repository" still
+// reached the user's terminal. stdout stays piped because we read it.
+const QUIET = ['ignore', 'pipe', 'ignore'];
+
 export class Store {
 	constructor({ root, pkg }) {
 		this.root = root;
@@ -66,7 +72,7 @@ export class Store {
 
 	// current HEAD — one cheap rev-parse per cache check vs a multi-thousand-file walk
 	gitHead() {
-		try { return execFileSync('git', ['rev-parse', 'HEAD'], { cwd: this.root }).toString().trim(); } catch { return 'no-git'; }
+		try { return execFileSync('git', ['rev-parse', 'HEAD'], { cwd: this.root, stdio: QUIET }).toString().trim(); } catch { return 'no-git'; }
 	}
 
 	ids(collection) {
@@ -128,7 +134,7 @@ export class Store {
 		const relPath = path.relative(this.root, file);
 		let previousContent;
 		try {
-			previousContent = execFileSync('git', ['show', `${hash}:${relPath}`], { cwd: this.root }).toString();
+			previousContent = execFileSync('git', ['show', `${hash}:${relPath}`], { cwd: this.root, stdio: QUIET }).toString();
 		} catch {
 			throw new Error(`${collection}/${id}: no content at ${hash} for ${relPath} — nothing was reverted.`);
 		}