@opengsd/gsd-core 1.4.4 → 1.5.0-rc.2

package/scripts/workflow-size.cjs

@@ -0,0 +1,90 @@
+'use strict';
+
+/**
+ * @file workflow-size.cjs
+ *
+ * Single source of truth for measuring workflow `.md` file sizes in bytes.
+ *
+ * Shared by `tests/workflow-size-budget.test.cjs` (the CI guard) and
+ * `scripts/update-size-baseline.cjs` (the baseline generator) so the two can
+ * never disagree on HOW a file is measured.  A divergence between the generator
+ * and the guard would silently mis-record the baseline (issue #1074).
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+const WORKFLOWS_DIR = path.join(__dirname, '..', 'gsd-core', 'workflows');
+
+/**
+ * Byte size of a file, counted as on an LF (Unix) checkout.
+ *
+ * The size budget is calibrated against `wc -c` on a Unix (LF) checkout, but
+ * these `.md` files have no `eol=lf` in `.gitattributes`, so Windows checks
+ * them out as CRLF.  Counting raw on-disk bytes there adds one byte per line,
+ * a Windows-only false positive that diverges from the LF calibration basis
+ * (issue #683).  Stripping CR yields the same LF byte count on every platform.
+ * This is still a raw byte count (not a trailing-newline-stripping line count).
+ *
+ * @param {string} filePath - Absolute or relative path to the file.
+ * @returns {number} LF-normalized byte length.
+ */
+function lfByteCount(filePath) {
+  const content = fs.readFileSync(filePath, 'utf-8');
+  return Buffer.byteLength(content.replace(/\r\n/g, '\n'), 'utf-8');
+}
+
+/**
+ * List top-level workflow stems (filenames without the `.md` extension), sorted.
+ * Non-recursive by design: per-mode bodies under `workflows/<name>/modes/` and
+ * templates are NOT measured — only the always-loaded top-level workflows.
+ *
+ * @param {string} [dir] - Workflows directory (defaults to the canonical one).
+ * @returns {string[]} Sorted stems, e.g. `['autonomous', 'plan-phase', ...]`.
+ */
+function listWorkflowStems(dir = WORKFLOWS_DIR) {
+  return fs
+    .readdirSync(dir)
+    .filter((f) => f.endsWith('.md'))
+    .map((f) => f.replace(/\.md$/, ''))
+    .sort();
+}
+
+/**
+ * Measure every top-level `.md` file in `dir`, keyed by filename, byte sizes.
+ * Generic over directory and an optional filename predicate — used for both
+ * workflows (`gsd-core/workflows/*.md`) and agents (`agents/gsd-*.md`) so the
+ * size guards and the baseline generator share one measurement path (#1074).
+ * Non-recursive by design.
+ *
+ * @param {string} dir - Directory to scan.
+ * @param {function(string): boolean} [predicate] - Filename filter (default: all `.md`).
+ * @returns {Object<string, number>} Map of filename → LF byte size, keys sorted.
+ */
+function measureMdFiles(dir, predicate = () => true) {
+  const out = {};
+  const names = fs
+    .readdirSync(dir)
+    .filter((f) => f.endsWith('.md') && predicate(f))
+    .sort();
+  for (const name of names) out[name] = lfByteCount(path.join(dir, name));
+  return out;
+}
+
+/**
+ * Measure every top-level workflow file, keyed by filename (`<stem>.md`).
+ *
+ * @param {string} [dir] - Workflows directory (defaults to the canonical one).
+ * @returns {Object<string, number>} Map of `<stem>.md` → LF byte size, sorted.
+ */
+function measureWorkflows(dir = WORKFLOWS_DIR) {
+  return measureMdFiles(dir);
+}
+
+module.exports = {
+  WORKFLOWS_DIR,
+  lfByteCount,
+  listWorkflowStems,
+  measureMdFiles,
+  measureWorkflows,
+};

package/scripts/run-cross-platform-tests.cjs

@@ -1,67 +0,0 @@
-'use strict';
-
-const { spawnSync } = require('child_process');
-const { ExitError, runMain } = require('./lib/cli-exit.cjs');
-
-const CROSS_PLATFORM_TEST_REASON = Object.freeze({
-  PASS: 'pass',
-  TEST_FAILURE: 'test_failure',
-  INFRA_FAILURE: 'infra_failure',
-  UNKNOWN_FAILURE: 'unknown_failure',
-});
-
-function classify(exitCode, output) {
-  if (exitCode === 0) return CROSS_PLATFORM_TEST_REASON.PASS;
-  if (exitCode === 2 || /infrastructure failure|worktree\.Construct/i.test(output)) {
-    return CROSS_PLATFORM_TEST_REASON.INFRA_FAILURE;
-  }
-  if (/\bFAIL\b|\d+\s+failures?\)/i.test(output)) {
-    return CROSS_PLATFORM_TEST_REASON.TEST_FAILURE;
-  }
-  return CROSS_PLATFORM_TEST_REASON.UNKNOWN_FAILURE;
-}
-
-function runCrossPlatformTests(options = {}, deps = {}) {
-  const {
-    base = 'next',
-    head = 'HEAD',
-    source = '.',
-    targets = 'linux,macos',
-    cwd = process.cwd(),
-  } = options;
-  const runner = deps.spawnSync || spawnSync;
-
-  const args = ['--targets', targets, '--base', base, '--head', head, '--source', source];
-  const result = runner('gsd-test', args, { cwd, encoding: 'utf8' });
-
-  const stdout = result.stdout || '';
-  const stderr = result.stderr || '';
-  const output = `${stdout}\n${stderr}`;
-  const exitCode = Number(result.status ?? 1);
-  const reason = classify(exitCode, output);
-
-  return {
-    ok: exitCode === 0,
-    reason,
-    exitCode,
-    command: ['gsd-test', ...args].join(' '),
-    stdout,
-    stderr,
-  };
-}
-
-if (require.main === module) {
-  function main() {
-    const result = runCrossPlatformTests();
-    const line = `[cross-platform-tests] reason=${result.reason} exit=${result.exitCode}`;
-    if (result.ok) {
-      process.stdout.write(`${line}\n`);
-      return 0;
-    }
-    process.stderr.write(`${line}\n`);
-    throw new ExitError(result.exitCode);
-  }
-  runMain(main);
-}
-
-module.exports = { CROSS_PLATFORM_TEST_REASON, runCrossPlatformTests };