@link-assistant/hive-mind 1.67.2 → 1.68.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @link-assistant/hive-mind
 
+## 1.68.0
+
+### Minor Changes
+
+- cbc7033: Switch the test runner to folder-based discovery and deprecate `start-screen` in favour of `--isolated screen` (issue #1758).
+  - `scripts/run-tests.mjs` now discovers every `*.mjs` / `*.test.mjs` / `*.test.js` file under `tests/` automatically. The hard-coded `LEGACY_DEFAULT_TESTS` allow-list is gone, so new test files no longer need a runner update to be picked up.
+  - New markers complement the existing `@hive-mind-test-suite <name>` marker:
+    - `@hive-mind-integration` — skip the file in the default suite; opt in via `--suite integration` or `HIVE_MIND_RUN_INTEGRATION=1`.
+    - `@hive-mind-test-skip` — exclude helper / fixture modules from every suite.
+  - `tests/integration-guard.mjs` exposes `skipUnlessIntegration(import.meta.url)` for token- or network-heavy tests.
+  - `src/start-screen.mjs` and `src/telegram-command-execution.lib.mjs::executeStartScreen` print a one-shot deprecation banner to stderr (suppressible with `HIVE_MIND_SUPPRESS_DEPRECATIONS=1`) recommending `--isolated screen`, which is already the default for `hive`/`solve` invocations through the Telegram bot.
+  - Adds regression tests `tests/test-issue-1758-runner-discovery.mjs`, `tests/test-issue-1758-start-screen-deprecation.mjs`, and `tests/test-issue-1758-integration-guard.mjs`.
+  - Documents the analysis under `docs/case-studies/issue-1758/`.
+
 ## 1.67.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.67.2",
+  "version": "1.68.0",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/start-screen.mjs

@@ -19,6 +19,24 @@ const printUsage = (log = console.error) => {
   }
 };
 
+/**
+ * Print a single-line deprecation notice to stderr the first time it is
+ * called per process. Suppressed when `HIVE_MIND_SUPPRESS_DEPRECATIONS=1`.
+ *
+ * Tracked via a module-scope flag (`deprecationWarned`) so a long-running
+ * process emits the banner only once even if `main()` is invoked multiple
+ * times in tests.
+ *
+ * @see https://github.com/link-assistant/hive-mind/issues/1758
+ */
+let deprecationWarned = false;
+const printDeprecationBanner = () => {
+  if (deprecationWarned) return;
+  if (process.env.HIVE_MIND_SUPPRESS_DEPRECATIONS === '1') return;
+  deprecationWarned = true;
+  console.error('⚠️  start-screen is deprecated; prefer `--isolated screen` (the default in newer hive/solve CLIs). Set HIVE_MIND_SUPPRESS_DEPRECATIONS=1 to silence this warning.');
+};
+
 const createStartScreenYargsConfig = yargsInstance =>
   yargsInstance
     .usage(START_SCREEN_USAGE[0])
@@ -288,6 +306,8 @@ async function createOrEnterScreen(sessionName, command, args, autoTerminate = f
 async function main() {
   const args = process.argv.slice(2);
 
+  printDeprecationBanner();
+
   if (args.includes('--help') || args.includes('-h')) {
     printUsage(console.log);
     process.exit(0);

package/src/telegram-command-execution.lib.mjs

@@ -4,6 +4,14 @@ import { exec as execCallback } from 'child_process';
 
 const exec = promisify(execCallback);
 
+let deprecationWarned = false;
+function warnStartScreenDeprecated() {
+  if (deprecationWarned) return;
+  if (process.env.HIVE_MIND_SUPPRESS_DEPRECATIONS === '1') return;
+  deprecationWarned = true;
+  console.warn('⚠️  executeStartScreen is deprecated; prefer the `--isolated screen` workflow exposed by hive/solve directly. Set HIVE_MIND_SUPPRESS_DEPRECATIONS=1 to silence this warning.');
+}
+
 async function findStartScreenCommand() {
   try {
     const { stdout } = await exec('which start-screen');
@@ -68,6 +76,8 @@ function executeWithCommand(startScreenCmd, command, args, verbose = false) {
 export async function executeStartScreen(command, args, options = {}) {
   const { verbose = false } = options;
 
+  warnStartScreenDeprecated();
+
   try {
     const whichPath = await findStartScreenCommand();