fix: resolve TODOs in client SDK filter comment and CLI test glob matching

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/cli/src/commands/test.ts

@@ -7,6 +7,51 @@ import fs from 'fs';
 import { QA as CoreQA } from '@objectstack/core';
 import { QA } from '@objectstack/spec';
 
+/**
+ * Resolve a glob-like pattern to matching file paths.
+ * Supports `*` (single segment wildcard) and `**` (recursive wildcard).
+ * Falls back to direct file path if no glob characters are present.
+ */
+function resolveGlob(pattern: string): string[] {
+  // Direct file path — no wildcards
+  if (!pattern.includes('*')) {
+    return fs.existsSync(pattern) ? [pattern] : [];
+  }
+
+  // Split pattern into the static base directory and the glob portion
+  const parts = pattern.split(path.sep.replace('\\', '/'));
+  // Also handle forward-slash on Windows
+  const segments = pattern.includes('/') ? pattern.split('/') : parts;
+
+  let baseDir = '.';
+  let globStart = 0;
+  for (let i = 0; i < segments.length; i++) {
+    if (segments[i].includes('*')) {
+      globStart = i;
+      break;
+    }
+    baseDir = i === 0 ? segments[i] : path.join(baseDir, segments[i]);
+  }
+
+  if (!fs.existsSync(baseDir)) return [];
+
+  // Convert the glob portion into a RegExp
+  const globPortion = segments.slice(globStart).join('/');
+  const regexStr = globPortion
+    .replace(/\./g, '\\.')           // escape dots
+    .replace(/\*\*\//g, '(.+/)?')   // ** matches any directory depth
+    .replace(/\*\*/g, '.*')         // trailing ** without slash
+    .replace(/\*/g, '[^/]*');       // * matches within a single segment
+  const regex = new RegExp(`^${regexStr}$`);
+
+  // Recursively read all files under baseDir
+  const entries = fs.readdirSync(baseDir, { recursive: true, encoding: 'utf-8' }) as string[];
+  return entries
+    .filter(entry => regex.test(entry.replace(/\\/g, '/')))
+    .map(entry => path.join(baseDir, entry))
+    .filter(fullPath => fs.statSync(fullPath).isFile());
+}
+
 export const testCommand = new Command('test')
   .description('Run Quality Protocol test scenarios against a running server')
   .argument('[files]', 'Glob pattern for test files (e.g. "qa/*.test.json")', 'qa/*.test.json')
@@ -21,23 +66,8 @@ export const testCommand = new Command('test')
     const adapter = new CoreQA.HttpTestAdapter(options.url, options.token);
     const runner = new CoreQA.TestRunner(adapter);
 
-    // 2. Find Files (Simple implementation for now)
-    // TODO: Use glob
-    const cwd = process.cwd();
-    const testFiles: string[] = [];
-    
-    // Very basic file finding for demo - assume explicit path or check local dir
-    if (fs.existsSync(filesPattern)) {
-        testFiles.push(filesPattern);
-    } else {
-        // Simple directory scan
-        const dir = path.dirname(filesPattern);
-        const ext = path.extname(filesPattern);
-        if (fs.existsSync(dir)) {
-            const files = fs.readdirSync(dir).filter(f => f.endsWith(ext) || f.endsWith('.json'));
-            files.forEach(f => testFiles.push(path.join(dir, f)));
-        }
-    }
+    // 2. Find test files using glob-style pattern matching
+    const testFiles: string[] = resolveGlob(filesPattern);
 
     if (testFiles.length === 0) {
         console.warn(chalk.yellow(`No test files found matching: ${filesPattern}`));

packages/client/src/index.ts

@@ -1060,8 +1060,9 @@ export class ObjectStackClient {
 
         // 4. Handle Filters (Simple vs AST)
         if (options.filters) {
-             // If looks like AST (not plain object map)
-             // TODO: robust check. safely assuming map for simplified find, and recommending .query() for AST
+             // Detect AST filter format vs simple key-value map. AST filters use an array structure
+             // with [field, operator, value] or [logicOp, ...nodes] shape (see isFilterAST).
+             // For complex filter expressions, use .query() which builds a proper QueryAST.
              if (this.isFilterAST(options.filters)) {
                  queryParams.set('filters', JSON.stringify(options.filters));
              } else {