@bastani/atomic 0.5.28 → 0.5.29-0

package/assets/settings.schema.json

@@ -17,11 +17,6 @@
       "enum": ["github", "azure-devops", "sapling"],
       "description": "Selected source control provider. On atomic startup, the corresponding GitHub / Azure DevOps MCP servers are enabled in `.claude/settings.json` and `.opencode/opencode.json`; the others are disabled to avoid unnecessary token consumption."
     },
-    "lastUpdated": {
-      "type": "string",
-      "format": "date-time",
-      "description": "ISO 8601 timestamp of the last configuration update."
-    },
     "providers": {
       "type": "object",
       "description": "Per-provider overrides for chatFlags and envVars. chatFlags replaces built-in defaults entirely when set; envVars are merged on top of defaults (user values win on conflict). Local .atomic/settings.json takes precedence over global ~/.atomic/settings.json.",

package/dist/services/config/atomic-config.d.ts

@@ -17,8 +17,6 @@ export declare function isScmProvider(value: unknown): value is ScmProvider;
 export interface AtomicConfig {
     /** Version of config schema */
     version?: number;
-    /** Timestamp of last init */
-    lastUpdated?: string;
     /** Selected source control provider (drives MCP server enable/disable sync). */
     scm?: ScmProvider;
     /** Per-provider overrides for chatFlags and envVars */

package/dist/services/config/atomic-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"atomic-config.d.ts","sourceRoot":"","sources":["../../../src/services/config/atomic-config.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,EAAE,KAAK,QAAQ,EAAE,KAAK,iBAAiB,EAAE,MAAM,YAAY,CAAC;AAOnE,0EAA0E;AAC1E,eAAO,MAAM,aAAa,gDAAiD,CAAC;AAC5E,MAAM,MAAM,WAAW,GAAG,CAAC,OAAO,aAAa,CAAC,CAAC,MAAM,CAAC,CAAC;AAEzD,wBAAgB,aAAa,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,WAAW,CAElE;AAED;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,+BAA+B;IAC/B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,6BAA6B;IAC7B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,gFAAgF;IAChF,GAAG,CAAC,EAAE,WAAW,CAAC;IAClB,uDAAuD;IACvD,SAAS,CAAC,EAAE,OAAO,CAAC,MAAM,CAAC,QAAQ,EAAE,iBAAiB,CAAC,CAAC,CAAC;CAC1D;AA0HD;;GAEG;AACH,wBAAsB,gBAAgB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,YAAY,GAAG,IAAI,CAAC,CAMvF;AAED;;GAEG;AACH,wBAAsB,gBAAgB,CACpC,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,OAAO,CAAC,YAAY,CAAC,GAC7B,OAAO,CAAC,IAAI,CAAC,CAsBf;AAED;;;;;;;GAOG;AACH,wBAAsB,oBAAoB,CACxC,QAAQ,EAAE,QAAQ,EAClB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,iBAAiB,CAAC,CAG5B"}
+{"version":3,"file":"atomic-config.d.ts","sourceRoot":"","sources":["../../../src/services/config/atomic-config.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,EAAE,KAAK,QAAQ,EAAE,KAAK,iBAAiB,EAAE,MAAM,YAAY,CAAC;AAOnE,0EAA0E;AAC1E,eAAO,MAAM,aAAa,gDAAiD,CAAC;AAC5E,MAAM,MAAM,WAAW,GAAG,CAAC,OAAO,aAAa,CAAC,CAAC,MAAM,CAAC,CAAC;AAEzD,wBAAgB,aAAa,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,WAAW,CAElE;AAED;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,+BAA+B;IAC/B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,gFAAgF;IAChF,GAAG,CAAC,EAAE,WAAW,CAAC;IAClB,uDAAuD;IACvD,SAAS,CAAC,EAAE,OAAO,CAAC,MAAM,CAAC,QAAQ,EAAE,iBAAiB,CAAC,CAAC,CAAC;CAC1D;AAuHD;;GAEG;AACH,wBAAsB,gBAAgB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,YAAY,GAAG,IAAI,CAAC,CAMvF;AAED;;GAEG;AACH,wBAAsB,gBAAgB,CACpC,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,OAAO,CAAC,YAAY,CAAC,GAC7B,OAAO,CAAC,IAAI,CAAC,CAqBf;AAED;;;;;;;GAOG;AACH,wBAAsB,oBAAoB,CACxC,QAAQ,EAAE,QAAQ,EAClB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,iBAAiB,CAAC,CAG5B"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bastani/atomic",
-  "version": "0.5.28",
+  "version": "0.5.29-0",
   "description": "Configuration management CLI and SDK for coding agents",
   "type": "module",
   "license": "MIT",
@@ -61,15 +61,12 @@
     "typecheck": "bunx tsc --noEmit && bunx tsc -p tests --noEmit",
     "lint": "oxlint --config=oxlint.json src",
     "lint:fix": "oxlint --config=oxlint.json --fix src",
-    "prepare": "lefthook install || true"
+    "prepare": "prek install -t pre-commit -t pre-push || true"
   },
-  "trustedDependencies": [
-    "lefthook"
-  ],
   "devDependencies": {
+    "@j178/prek": "^0.3.10",
     "@types/bun": "^1.3.12",
     "@types/react": "^19.2.14",
-    "lefthook": "^2.1.6",
     "oxlint": "^1.61.0",
     "typescript": "^6.0.3",
     "typescript-language-server": "^5.1.3"

package/src/cli.ts

@@ -404,6 +404,12 @@ export const program = createProgram();
  */
 async function main(): Promise<void> {
     try {
+        // Bootstrap `~/.atomic/settings.json` on every invocation if absent,
+        // so users always have a file to edit with JSON Schema intellisense
+        // wired up. Idempotent; swallows FS errors internally.
+        const { ensureGlobalAtomicSettings } = await import("./services/config/settings.ts");
+        await ensureGlobalAtomicSettings();
+
         // Sync tooling deps and global skills on first launch after install
         // or upgrade. Runs at most once per version bump (gated on a marker
         // file under ~/.atomic). Skipped for `--version` / `--help` so info

package/src/services/config/atomic-config.ts

@@ -30,8 +30,6 @@ export function isScmProvider(value: unknown): value is ScmProvider {
 export interface AtomicConfig {
   /** Version of config schema */
   version?: number;
-  /** Timestamp of last init */
-  lastUpdated?: string;
   /** Selected source control provider (drives MCP server enable/disable sync). */
   scm?: ScmProvider;
   /** Per-provider overrides for chatFlags and envVars */
@@ -97,10 +95,8 @@ function pickAtomicConfig(record: JsonRecord | null): AtomicConfig | null {
 
   const config: AtomicConfig = {};
   const version = record.version;
-  const lastUpdated = record.lastUpdated;
 
   if (typeof version === "number") config.version = version;
-  if (typeof lastUpdated === "string") config.lastUpdated = lastUpdated;
   if (isScmProvider(record.scm)) config.scm = record.scm;
 
   const providers = pickProviders(record.providers);
@@ -144,7 +140,6 @@ function mergeConfigs(...configs: Array<AtomicConfig | null>): AtomicConfig | nu
   for (const config of configs) {
     if (!config) continue;
     if (config.version !== undefined) merged.version = config.version;
-    if (config.lastUpdated !== undefined) merged.lastUpdated = config.lastUpdated;
     if (config.scm !== undefined) merged.scm = config.scm;
 
     if (config.providers) {
@@ -186,7 +181,6 @@ export async function saveAtomicConfig(
     ...currentConfig,
     ...updates,
     version: 1,
-    lastUpdated: new Date().toISOString(),
   };
 
   const nextSettings: JsonRecord = {

package/src/services/config/settings.ts

@@ -20,7 +20,6 @@ import type { ScmProvider } from "./atomic-config.ts";
 interface AtomicSettings {
   $schema?: string;
   version?: number;
-  lastUpdated?: string;
   telemetryEnabled?: boolean;
   scm?: ScmProvider;
   providers?: Partial<Record<AgentKey, ProviderOverrides>>;
@@ -54,6 +53,25 @@ async function writeGlobalSettings(settings: AtomicSettings): Promise<void> {
   await Bun.write(path, JSON.stringify(settings, null, 2));
 }
 
+/**
+ * Ensure `~/.atomic/settings.json` exists. Called once at CLI startup so
+ * users have a valid file to edit (with JSON Schema intellisense wired up
+ * via `$schema`) without having to run any explicit init command.
+ *
+ * Idempotent — no-op if the file already exists. Best-effort: filesystem
+ * errors (e.g. read-only home) are swallowed so the CLI never blocks on
+ * this side-effect.
+ */
+export async function ensureGlobalAtomicSettings(): Promise<void> {
+  try {
+    const path = globalSettingsPath();
+    if (await Bun.file(path).exists()) return;
+    await writeGlobalSettings({ version: 1 });
+  } catch (e) {
+    console.warn(`[settings] failed to bootstrap global settings: ${errorMessage(e)}`);
+  }
+}
+
 /**
  * Set telemetry enabled/disabled in global settings.
  */

package/src/services/system/auth.test.ts

@@ -13,8 +13,13 @@ import {
   test,
   expect,
   beforeEach,
+  beforeAll,
+  afterAll,
   mock,
 } from "bun:test";
+import { chmodSync, mkdtempSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
 
 // ─── Copilot SDK fake ──────────────────────────────────────────────────────
 // `CopilotClient` is a class; the constructor captures latest test state.
@@ -45,10 +50,6 @@ class FakeCopilotClient {
   }
 }
 
-mock.module("@github/copilot-sdk", () => ({
-  CopilotClient: FakeCopilotClient,
-}));
-
 // ─── Claude Agent SDK fake ────────────────────────────────────────────────
 // `query()` returns something with `initializationResult()` and `close()`.
 // We ignore the `prompt` stream — the real SDK consumes it lazily, and the
@@ -65,17 +66,50 @@ let claudeInit = mock<() => Promise<{ account: ClaudeAccount }>>(async () => ({
 }));
 let claudeClose = mock(() => {});
 
-mock.module("@anthropic-ai/claude-agent-sdk", () => ({
-  query: () => ({
-    initializationResult: () => claudeInit(),
-    close: () => claudeClose(),
-  }),
-}));
+// `mock.module` is process-global in Bun and leaks across every test file
+// loaded in the same run — live ESM bindings in other files rebind to the
+// stub as soon as it registers, and re-registering with the real module in
+// `afterAll` does not restore the original namespace identity. Capture the
+// real SDK modules first, install the mocks only while this file's tests
+// are running, and never mock `claude.ts` (other test files exercise its
+// real exports). All consumers in `auth.ts` use dynamic `await import(...)`,
+// so lazy mock registration is safe here.
+const actualCopilotSdk = await import("@github/copilot-sdk");
+const actualClaudeSdk = await import("@anthropic-ai/claude-agent-sdk");
+
+// Put a fake `claude` binary on PATH so `resolveHeadlessClaudeBin()` (called
+// by `checkClaudeAuth`) succeeds without hitting the real CLI on disk. The
+// mocked SDK `query()` never actually spawns the subprocess — the path is
+// only passed through to the SDK constructor.
+let pathBefore: string | undefined;
+
+beforeAll(() => {
+  const dir = mkdtempSync(join(tmpdir(), "atomic-auth-test-path-"));
+  const bin = join(dir, "claude");
+  writeFileSync(bin, "#!/usr/bin/env sh\nexit 0\n");
+  chmodSync(bin, 0o755);
+  pathBefore = process.env.PATH;
+  process.env.PATH = `${dir}:${process.env.PATH ?? ""}`;
+
+  mock.module("@github/copilot-sdk", () => ({
+    ...actualCopilotSdk,
+    CopilotClient: FakeCopilotClient,
+  }));
+  mock.module("@anthropic-ai/claude-agent-sdk", () => ({
+    ...actualClaudeSdk,
+    query: () => ({
+      initializationResult: () => claudeInit(),
+      close: () => claudeClose(),
+    }),
+  }));
+});
 
-// Stub the claude provider module so we don't probe PATH for `claude`.
-mock.module("../../sdk/providers/claude.ts", () => ({
-  resolveHeadlessClaudeBin: () => "/usr/local/bin/claude",
-}));
+afterAll(() => {
+  if (pathBefore === undefined) delete process.env.PATH;
+  else process.env.PATH = pathBefore;
+  mock.module("@github/copilot-sdk", () => ({ ...actualCopilotSdk }));
+  mock.module("@anthropic-ai/claude-agent-sdk", () => ({ ...actualClaudeSdk }));
+});
 
 const { checkAgentAuth } = await import("./auth.ts");