feat: add asset filename filtering to improve asset collection

- Introduced a new utility function to ignore specific asset filenames that are not real app assets, such as system files (.DS_Store, Thumbs.db).
- Updated asset collection logic in both appCollector and pullAssets to filter out ignored filenames.
- Added tests to ensure that ignored files are correctly excluded from the asset collection process.

Author: sharjeelyunus

src/core/appCollector.ts

@@ -8,6 +8,7 @@ import path from 'path';
 
 import type { ArtifactProp } from './artifacts.js';
 import { ARTIFACT_CONFIGS } from './artifacts.js';
+import { isIgnoredAssetFileName } from './assetIgnore.js';
 import { processWithConcurrency } from './concurrency.js';
 
 export interface ParsedAppFiles {
@@ -45,6 +46,7 @@ async function collectAssetBasenames(rootDir: string): Promise<string[]> {
     return entries
       .filter((e) => e.isFile())
       .map((e) => e.name)
+      .filter((name) => !isIgnoredAssetFileName(name))
       .sort();
   } catch {
     return [];

src/core/assetIgnore.ts

@@ -0,0 +1,20 @@
+/**
+ * Asset filenames that should never be treated as real app assets.
+ * These are commonly created by OS tooling and can accidentally appear in `assets/`.
+ */
+export function isIgnoredAssetFileName(fileName: string): boolean {
+  const name = fileName.trim();
+  if (name === '') return true;
+
+  // macOS Finder metadata
+  if (name === '.DS_Store') return true;
+
+  // AppleDouble sidecar files (e.g. when copying to non-HFS filesystems)
+  if (name.startsWith('._')) return true;
+
+  // Windows metadata
+  if (name === 'Thumbs.db') return true;
+  if (name === 'desktop.ini') return true;
+
+  return false;
+}

src/core/pullAssets.ts

@@ -3,6 +3,7 @@ import path from 'path';
 
 import type { AssetDTO } from './dto.js';
 import type { upsertEnvConfig } from './envConfig.js';
+import { isIgnoredAssetFileName } from './assetIgnore.js';
 
 export class AssetPullError extends Error {
   constructor(
@@ -210,7 +211,10 @@ export async function applyCloudAssetsToFs(params: {
   let localFiles: string[] = [];
   try {
     const entries = await fs.readdir(assetsDir, { withFileTypes: true });
-    localFiles = entries.filter((e) => e.isFile()).map((e) => e.name);
+    localFiles = entries
+      .filter((e) => e.isFile())
+      .map((e) => e.name)
+      .filter((name) => !isIgnoredAssetFileName(name));
   } catch {
     localFiles = [];
   }

tests/core/appCollector.test.ts

@@ -24,6 +24,17 @@ describe('collectAppFiles', () => {
     expect(result.assetFiles).toEqual(['logo.png']);
   });
 
+  it('excludes .DS_Store and AppleDouble files from assets', async () => {
+    await fs.mkdir(path.join(tmpDir, 'assets'), { recursive: true });
+    await fs.writeFile(path.join(tmpDir, 'assets', '.DS_Store'), Buffer.from([1]));
+    await fs.writeFile(path.join(tmpDir, 'assets', '._logo.png'), Buffer.from([2]));
+    await fs.writeFile(path.join(tmpDir, 'assets', 'logo.png'), Buffer.from([3]));
+
+    const result = await collectAppFiles(tmpDir);
+
+    expect(result.assetFiles).toEqual(['logo.png']);
+  });
+
   it('collects screens, widgets, scripts', async () => {
     await fs.mkdir(path.join(tmpDir, 'screens'), { recursive: true });
     await fs.mkdir(path.join(tmpDir, 'widgets'), { recursive: true });