sneakoscope 0.7.74 → 0.7.78

package/src/core/version-manager.mjs

@@ -1,22 +1,38 @@
 import path from 'node:path';
 import fsp from 'node:fs/promises';
-import { ensureDir, exists, nowIso, readJson, runProcess, writeJsonAtomic, writeTextAtomic } from './fsx.mjs';
+import { exists, nowIso, readJson, runProcess, writeJsonAtomic, writeTextAtomic } from './fsx.mjs';
 
 const VERSION_HOOK_MARKER = 'Sneakoscope Codex Version Guard';
 const VERSION_STATE_FILE = 'sks-version-state.json';
 const DEFAULT_BUMP = 'patch';
 
 export async function installVersionGitHook(root, commandPrefix = 'sks') {
+  void root;
+  void commandPrefix;
+  return {
+    ok: false,
+    installed: false,
+    reason: 'pre_commit_hooks_unsupported',
+    message: 'SKS no longer installs Git pre-commit hooks. Use `sks versioning bump` and release checks explicitly.'
+  };
+}
+
+export async function disableVersionGitHook(root) {
+  await setVersionPolicyEnabled(root, false);
   const git = await gitPaths(root);
-  if (!git.ok) return { ok: true, installed: false, reason: git.reason || 'not_git' };
-  const hookPath = git.hook_path;
-  const block = versionHookBlock(commandPrefix);
-  const current = await readFileMaybe(hookPath);
-  const next = mergeShellBlock(current, VERSION_HOOK_MARKER, block);
-  await ensureDir(path.dirname(hookPath));
-  await writeTextAtomic(hookPath, next);
-  await fsp.chmod(hookPath, 0o755).catch(() => {});
-  return { ok: true, installed: true, hook_path: hookPath };
+  if (!git.ok) return { ok: true, disabled: true, hook_removed: false, reason: git.reason || 'not_git' };
+  const current = await readFileMaybe(git.hook_path);
+  if (!current.includes(`BEGIN ${VERSION_HOOK_MARKER}`)) {
+    return { ok: true, disabled: true, hook_removed: false, hook_path: git.hook_path, reason: 'managed_hook_not_installed' };
+  }
+  const next = removeShellBlock(current, VERSION_HOOK_MARKER);
+  if (next.trim() === '#!/bin/sh' || next.trim() === '#!/usr/bin/env sh' || !next.trim()) {
+    await fsp.rm(git.hook_path, { force: true });
+    return { ok: true, disabled: true, hook_removed: true, hook_path: git.hook_path };
+  }
+  await writeTextAtomic(git.hook_path, next);
+  await fsp.chmod(git.hook_path, 0o755).catch(() => {});
+  return { ok: true, disabled: true, hook_removed: true, hook_path: git.hook_path };
 }
 
 export async function versioningStatus(root) {
@@ -40,7 +56,7 @@ export async function versioningStatus(root) {
     state_path: path.join(git.common_dir, VERSION_STATE_FILE),
     last_version: state.last_version || null,
     runtime_drift: runtimeDrift,
-    reason: version ? null : 'package_json_version_missing'
+    reason: !policy.enabled ? 'disabled_by_policy' : (version ? null : 'package_json_version_missing')
   };
 }
 
@@ -190,11 +206,38 @@ export async function verifyProjectVersion(root, opts = {}) {
 async function versionPolicy(root) {
   const policy = await readJson(path.join(root, '.sneakoscope', 'policy.json'), {});
   return {
-    enabled: policy.versioning?.enabled !== false,
+    enabled: policy.versioning?.enabled === true,
     bump: policy.versioning?.bump || DEFAULT_BUMP
   };
 }
 
+async function setVersionPolicyEnabled(root, enabled) {
+  const policyPath = path.join(root, '.sneakoscope', 'policy.json');
+  const policy = await readJson(policyPath, {});
+  await writeJsonAtomic(policyPath, {
+    ...policy,
+    git: {
+      ...(policy.git || {}),
+      versioning: {
+        ...(policy.git?.versioning || {}),
+        enabled: Boolean(enabled),
+        bump: policy.git?.versioning?.bump || policy.versioning?.bump || DEFAULT_BUMP,
+        lock: 'git-common-dir/sks-version.lock',
+        state: 'git-common-dir/sks-version-state.json'
+      }
+    },
+    versioning: {
+      ...(policy.versioning || {}),
+      enabled: Boolean(enabled),
+      bump: policy.versioning?.bump || DEFAULT_BUMP,
+      trigger: 'manual',
+      lock_scope: 'git-common-dir',
+      managed_files: policy.versioning?.managed_files || ['package.json', 'package-lock.json', 'npm-shrinkwrap.json'],
+      collision_policy: policy.versioning?.collision_policy || 'explicit_bump_only'
+    }
+  });
+}
+
 async function gitPaths(root) {
   const top = await git(root, ['rev-parse', '--show-toplevel']);
   if (top.code !== 0) return { ok: false, reason: 'not_git' };
@@ -287,7 +330,7 @@ async function syncChangelogVersionSection(root, version) {
     text = text.replace(/^#\s+Changelog\s*$/m, (title) => `${title}\n\n## [Unreleased]`);
   }
 
-  const managedSection = `\n## [${version}] - ${date}\n\n### Fixed\n\n- Keep release metadata aligned after the automatic SKS version guard advances the package version.\n`;
+  const managedSection = `\n## [${version}] - ${date}\n\n### Fixed\n\n- Keep release metadata aligned after an explicit SKS version bump advances the package version.\n`;
   const next = text.replace(/^##\s+\[Unreleased\]\s*$/m, (heading) => `${heading}\n${managedSection}`);
   if (next === text) return { files: [], relative_files: [] };
   await writeTextAtomic(file, next);
@@ -341,26 +384,13 @@ function bumpSemver(v, bump = DEFAULT_BUMP) {
   return { major: v.major, minor: v.minor, patch: v.patch + 1 };
 }
 
-function versionHookBlock(commandPrefix) {
-  return `# SKS keeps package versions unique across worker commits.\n${commandPrefix} versioning pre-commit\nstatus=$?\nif [ $status -ne 0 ]; then\n  echo \"SKS versioning blocked commit. Run: sks versioning status\" >&2\n  exit $status\nfi`;
-}
-
-function mergeShellBlock(current, marker, block) {
+function removeShellBlock(current, marker) {
   const begin = `# BEGIN ${marker}`;
   const end = `# END ${marker}`;
-  const managed = `${begin}\n${block.trim()}\n${end}\n`;
-  if (!current.trim()) return `#!/bin/sh\n${managed}`;
-  const withShebang = current.startsWith('#!') ? current : `#!/bin/sh\n${current}`;
-  const beginIdx = withShebang.indexOf(begin);
-  const endIdx = withShebang.indexOf(end);
-  if (beginIdx >= 0 && endIdx >= beginIdx) {
-    return `${withShebang.slice(0, beginIdx)}${managed}${withShebang.slice(endIdx + end.length).replace(/^\n/, '')}`;
-  }
-  const lines = withShebang.split('\n');
-  if (lines[0]?.startsWith('#!')) {
-    return `${lines[0]}\n${managed}${lines.slice(1).join('\n').replace(/^\n/, '')}`.replace(/\s*$/, '\n');
-  }
-  return `${managed}${withShebang.replace(/^\n/, '').replace(/\s*$/, '\n')}`;
+  const beginIdx = current.indexOf(begin);
+  const endIdx = current.indexOf(end);
+  if (beginIdx < 0 || endIdx < beginIdx) return current;
+  return `${current.slice(0, beginIdx)}${current.slice(endIdx + end.length).replace(/^\n/, '')}`.replace(/\s*$/, '\n');
 }
 
 async function readFileMaybe(file) {