@link-assistant/hive-mind 2.4.1 → 2.5.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @link-assistant/hive-mind
 
+## 2.5.1
+
+### Patch Changes
+
+- 4054bcf: Commit the `--development-log` artifacts before signalling pull request readiness (issue #2048), so the development-log commit is part of the diff that CI and `--auto-restart-until-mergeable` evaluate. Previously the log commit landed after the "Ready to merge" signal and could break CI (e.g. a missing changeset) with no readiness re-evaluation.
+
+## 2.5.0
+
+### Minor Changes
+
+- 152f55a: Add generation-relative `sol`, `terra`, and `luna` Codex aliases and accept `openai/` and `openai.` prefixes for every known OpenAI model.
+
 ## 2.4.1
 
 ### Patch Changes

package/package.json

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

package/src/development-log.lib.mjs

@@ -234,6 +234,9 @@ export const collectAndCommitDevelopmentLogArtifacts = async ({ enabled, reposit
     return { skipped: 'missing-repository-path' };
   }
 
+  // Issue #2048: verbose trace so the commit timing (relative to PR readiness signals) is diagnosable from logs.
+  await log?.(`🔍 Development log finalize: issue #${issueNumber ?? '?'}, PR #${prNumber ?? 'pending'}, branch ${branchName ?? 'none'}, session ${sessionId ?? 'none'}`, { verbose: true });
+
   try {
     const artifacts = await writeDevelopmentLogArtifacts({
       repositoryPath,

package/src/models/index.mjs

@@ -151,6 +151,47 @@ export const codexModels = {
   'gpt-4o': 'gpt-4o',
 };
 
+const CODEX_GENERATION_ALIAS_PATTERN = /^gpt-(\d+(?:\.\d+)?)-(sol|terra|luna)$/;
+const OPENAI_MODEL_PREFIX_PATTERN = /^openai([/.])/;
+
+/**
+ * Resolve sol/terra/luna to the newest generation that contains the complete
+ * alias family. A complete family prevents a partially rolled-out catalog from
+ * moving only some aliases to a newer generation.
+ */
+export const getLatestCodexGenerationAliases = (models = codexModels) => {
+  const generations = new Map();
+
+  for (const modelId of Object.values(models)) {
+    const bareModelId = modelId.replace(OPENAI_MODEL_PREFIX_PATTERN, '');
+    const match = bareModelId.match(CODEX_GENERATION_ALIAS_PATTERN);
+    if (!match) continue;
+
+    const [, generation, alias] = match;
+    if (!generations.has(generation)) generations.set(generation, {});
+    generations.get(generation)[alias] = bareModelId;
+  }
+
+  const latestCompleteGeneration = [...generations.entries()].filter(([, aliases]) => ['sol', 'terra', 'luna'].every(alias => aliases[alias])).sort(([left], [right]) => right.localeCompare(left, undefined, { numeric: true }))[0];
+
+  return latestCompleteGeneration?.[1] || {};
+};
+
+const getCodexModelVariants = () => {
+  const bareModels = [...new Set(Object.values(codexModels).map(modelId => modelId.replace(OPENAI_MODEL_PREFIX_PATTERN, '')))];
+  const aliases = getLatestCodexGenerationAliases();
+  const variants = { ...codexModels, ...aliases };
+
+  for (const [name, modelId] of Object.entries({ ...Object.fromEntries(bareModels.map(modelId => [modelId, modelId])), ...aliases })) {
+    variants[`openai/${name}`] = `openai/${modelId}`;
+    variants[`openai.${name}`] = `openai.${modelId}`;
+  }
+
+  return variants;
+};
+
+export const CODEX_MODEL_VARIANTS = getCodexModelVariants();
+
 // Qwen Code models
 export const qwenModels = {
   qwen: 'qwen3-coder-plus',
@@ -269,7 +310,7 @@ export const OPENCODE_MODELS = {
 };
 
 export const CODEX_MODELS = {
-  ...codexModels,
+  ...CODEX_MODEL_VARIANTS,
   'gpt-5': 'gpt-5',
   'gpt-5.5': 'gpt-5.5',
   'gpt-5.5-mini': 'gpt-5.5-mini',
@@ -346,7 +387,7 @@ export const getModelMapForTool = tool => {
     case 'opencode':
       return opencodeModels;
     case 'codex':
-      return codexModels;
+      return CODEX_MODEL_VARIANTS;
     case 'gemini':
       return geminiModels;
     case 'qwen':
@@ -429,7 +470,7 @@ export const mapModelForTool = (tool, model) => {
     case 'opencode':
       return opencodeModels[model] || model;
     case 'codex':
-      return codexModels[model] || model;
+      return CODEX_MODEL_VARIANTS[model] || model;
     case 'gemini':
       return geminiModels[model] || model;
     case 'qwen':
@@ -456,7 +497,7 @@ export const isModelCompatibleWithTool = (tool, model) => {
     case 'opencode':
       return mappedModel.includes('/') || Object.keys(opencodeModels).includes(model);
     case 'codex':
-      return Object.keys(codexModels).includes(model) || mappedModel.startsWith('gpt-');
+      return Object.hasOwn(CODEX_MODEL_VARIANTS, model);
     case 'gemini':
       return Object.keys(geminiModels).includes(model) || mappedModel.startsWith('gemini-');
     case 'qwen':
@@ -480,7 +521,7 @@ export const getValidModelsForTool = tool => {
     case 'opencode':
       return Object.keys(opencodeModels);
     case 'codex':
-      return Object.keys(codexModels);
+      return Object.keys(CODEX_MODEL_VARIANTS);
     case 'gemini':
       return Object.keys(geminiModels);
     case 'qwen':

package/src/solve.mjs

@@ -1227,6 +1227,8 @@ try {
 
   await enforceRequestedBaseBranch();
 
+  // Issue #2048: commit+push dev log BEFORE any PR readiness signal so its CI gates readiness (was last, breaking CI post-signal; PR #2046, docs/case-studies/issue-2048). Idempotent: trailing call is a no-op. prettier-ignore
+  await finalizeDevelopmentLog();
   // Issue #1263 / #1728: Working session summary attachment.
   // Routed through the shared maybeAttachWorkingSessionSummary helper so that
   // top-level solve, auto-restart-until-mergeable, and watch-mode iterations
@@ -1466,7 +1468,7 @@ try {
   // Issue #1516: Cleanup after all signals (was before verifyResults, caused premature commits)
   await cleanupClaudeFile(tempDir, branchName, claudeCommitHash, argv);
 
-  await finalizeDevelopmentLog(); // Issue #1596: preserve session before ending work.
+  await finalizeDevelopmentLog(); // Issue #1596/#2048: idempotent no-op on the success path (already committed before readiness signal); still preserves late/error work.
   await endWorkSession({ isContinueMode, prNumber, argv, log, formatAligned, $, logsAttached });
 } catch (error) {
   await finalizeDevelopmentLog(); // Preserve failed/interrupted sessions too.