atris 3.34.0 → 3.35.0

package/utils/update-check.js

@@ -5,6 +5,7 @@ const os = require('os');
 const { spawn, spawnSync } = require('child_process');
 
 const PACKAGE_NAME = 'atris';
+const NPM_SELF_UPDATE_COMMAND = `npm install -g ${PACKAGE_NAME}@latest`;
 const CHECK_INTERVAL_MS = 60 * 60 * 1000; // 1 hour
 const ATRIS_DIR = path.join(os.homedir(), '.atris');
 const CACHE_FILE = path.join(ATRIS_DIR, '.update-check');
@@ -217,13 +218,31 @@ async function checkForUpdates(force = false) {
   return null;
 }
 
-function showUpdateNotification(updateInfo) {
+function isGitCheckout(packageRoot = path.join(__dirname, '..')) {
+  return fs.existsSync(path.join(path.resolve(packageRoot), '.git'));
+}
+
+function getNpmSelfUpdateCommand() {
+  return NPM_SELF_UPDATE_COMMAND;
+}
+
+function getNpmSelfUpdateSpawnArgs() {
+  return {
+    command: 'npm',
+    args: ['install', '-g', `${PACKAGE_NAME}@latest`],
+  };
+}
+
+function showUpdateNotification(updateInfo, options = {}) {
   if (!updateInfo || !updateInfo.needsUpdate) return;
 
-  // Single yellow warning line — non-intrusive
+  const packageRoot = options.packageRoot || path.join(__dirname, '..');
   const yellow = '\x1b[33m';
   const reset = '\x1b[0m';
-  console.log(`${yellow}Update available: ${updateInfo.installed} → ${updateInfo.latest}. Run: atris upgrade${reset}`);
+  const installHint = isGitCheckout(packageRoot)
+    ? 'run: atris upgrade'
+    : `run: ${NPM_SELF_UPDATE_COMMAND}`;
+  console.log(`${yellow}update available: ${updateInfo.installed} -> ${updateInfo.latest}. ${installHint}${reset}`);
 }
 
 function inspectInstallGitState(packageRoot = path.join(__dirname, '..')) {
@@ -288,9 +307,8 @@ function shouldAutoUpdate(updateInfo, state, env = process.env) {
   if (mode === 'off') return false;
   if (mode === 'force') return true;
 
-  // Packaged npm installs are not git repositories. Git checkouts may be linked
-  // dev installs, where a global npm install would not update the code on PATH.
-  return !(state && state.isGitRepo);
+  const packageRoot = state && state.root ? state.root : path.join(__dirname, '..');
+  return !isGitCheckout(packageRoot);
 }
 
 function autoUpdate(updateInfo, options = {}) {
@@ -331,4 +349,7 @@ module.exports = {
   shouldAutoUpdate,
   inspectInstallGitState,
   formatInstallGitWarning,
+  isGitCheckout,
+  getNpmSelfUpdateCommand,
+  getNpmSelfUpdateSpawnArgs,
 };

package/atris/learnings.jsonl

@@ -1 +0,0 @@
-{"ts":"2026-05-08T02:32:38.257Z","type":"pitfall","key":"narrow-grep-hides-matches","insight":"When operator says X exists in our code and a single-pattern grep returns 0 hits, do NOT conclude it does not exist. Run wider grep -rn first (whole backend/, not one subdir) before doubting. Tonight 2026-05-08: grepped \"gpt-5\\.5|gpt5\\.5|gpt-5p5\" in clients/ only, missed 20+ matches in routers/atris2_router.py + tests/, told operator gpt-5.5 was not wired. Operator was right.","confidence":5,"source":"observed","files":[]}