atris 3.15.13 → 3.15.22

package/lib/manifest.js

@@ -52,7 +52,7 @@ function buildManifest(files, commitHash, metadata = {}) {
     last_sync: new Date().toISOString(),
     last_commit: commitHash || null,
     workspace_root: metadata.workspaceRoot || null,
-    files,
+    files: filterSyncFiles(files),
   };
 }
 
@@ -62,11 +62,33 @@ function buildManifest(files, commitHash, metadata = {}) {
  */
 const SKIP_DIRS = new Set([
   'node_modules', '__pycache__', '.git', 'venv', '.venv',
-  'lost+found', '.cache', '.atris',
+  'lost+found', '.cache', '.atris', '.claude', '.cursor', '.next',
+  '.vscode', 'dist', 'build', 'coverage', 'tmp', 'temp',
   // Defense-in-depth: macOS/system dirs that should never be scanned as
   // part of any atris workspace, even if outputDir is accidentally wide.
   'Library', 'Applications', 'System',
 ]);
+const SYNC_ALLOWED_DOT_DIRS = new Set(['.github']);
+
+function isIgnoredSyncPath(filePath) {
+  const parts = String(filePath || '').replace(/\\/g, '/').split('/').filter(Boolean);
+  return parts.some((part, index) => {
+    if (SKIP_DIRS.has(part)) return true;
+    return index < parts.length - 1 && part.startsWith('.') && !SYNC_ALLOWED_DOT_DIRS.has(part);
+  });
+}
+
+function filterSyncFiles(files = {}) {
+  return Object.fromEntries(
+    Object.entries(files || {}).filter(([filePath]) => !isIgnoredSyncPath(filePath))
+  );
+}
+
+function isSyncTextBuffer(buffer) {
+  if (!Buffer.isBuffer(buffer)) return false;
+  if (buffer.includes(0)) return false;
+  return Buffer.from(buffer.toString('utf8'), 'utf8').equals(buffer);
+}
 
 function computeLocalHashes(localDir) {
   const files = {};
@@ -79,17 +101,19 @@ function computeLocalHashes(localDir) {
       return;
     }
     for (const entry of entries) {
-      if (entry.name.startsWith('.')) continue;
       const fullPath = path.join(dir, entry.name);
       const relFromRoot = path.relative(localDir, fullPath);
       if (entry.isDirectory()) {
         if (SKIP_DIRS.has(entry.name)) continue;
+        if (entry.name.startsWith('.') && !SYNC_ALLOWED_DOT_DIRS.has(entry.name)) continue;
         walk(fullPath);
       } else if (entry.isFile()) {
+        if (entry.name.startsWith('.')) continue;
         const relPath = '/' + path.relative(localDir, fullPath);
         try {
           // Hash raw bytes to match warm runner's _hash_bytes(data)
           const rawBytes = fs.readFileSync(fullPath);
+          if (!isSyncTextBuffer(rawBytes)) continue;
           const hash = crypto.createHash('sha256').update(rawBytes).digest('hex');
           files[relPath] = { hash, size: rawBytes.length };
         } catch {
@@ -114,6 +138,9 @@ function computeLocalHashes(localDir) {
  * Each array contains file path strings.
  */
 function threeWayCompare(localFiles, remoteFiles, manifest) {
+  localFiles = filterSyncFiles(localFiles);
+  remoteFiles = filterSyncFiles(remoteFiles);
+
   const result = {
     toPull: [],
     toPush: [],
@@ -147,7 +174,7 @@ function threeWayCompare(localFiles, remoteFiles, manifest) {
     return result;
   }
 
-  const base = manifest.files;
+  const base = filterSyncFiles(manifest.files);
   const allPaths = new Set([
     ...Object.keys(localFiles),
     ...Object.keys(remoteFiles),
@@ -226,4 +253,7 @@ module.exports = {
   threeWayCompare,
   getManifestPath,
   SKIP_DIRS,
+  isIgnoredSyncPath,
+  filterSyncFiles,
+  isSyncTextBuffer,
 };

package/lib/scorecard.js

@@ -280,9 +280,10 @@ function detectEndgameCompletion(atrisDir) {
 
   const slug = endgameSectionMatch[1];
 
-  // Check if there are any endgame-tagged tasks in backlog or in-progress
+  // Check if there are any endgame-tagged tasks still before acceptance.
   const hasActiveEndgame = todo.backlog.some(t => t.tag === 'endgame')
-    || todo.inProgress.some(t => t.tag === 'endgame');
+    || todo.inProgress.some(t => t.tag === 'endgame')
+    || (todo.review || []).some(t => t.tag === 'endgame');
 
   return {
     complete: !hasActiveEndgame,