fix: CLI --days flag was silently ignored, daily chart never rendered

The CLI sent ?days=N to the API but the core only reads ?since=.
Users always got 7 days of data regardless of --days value. Also,
the stats daily chart checked data.daily but the API returns
data.timeSeries, so the chart section was permanently dead code.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: dannyshmueli

bin/cli.mjs

@@ -188,10 +188,10 @@ const cmdStats = withApi(async (api, project, days = 7) => {
     }
   }
 
-  if (data.daily && data.daily.length > 0) {
+  if (data.timeSeries && data.timeSeries.length > 0) {
     log('');
     heading('Daily:');
-    for (const d of data.daily) {
+    for (const d of data.timeSeries) {
       const bar = '█'.repeat(Math.min(Math.ceil(d.total_events / 5), 40));
       log(`  ${d.date}  ${GREEN}${bar}${RESET}  ${d.total_events} events`);
     }

lib/api.mjs

@@ -83,15 +83,18 @@ export class AgentAnalyticsAPI {
 
   // Stats
   async getStats(project, days = 7, { returnHeaders = false } = {}) {
-    return this.request('GET', `/stats?${this._qs({ project, days })}`, undefined, { returnHeaders });
+    const since = `${days}d`;
+    return this.request('GET', `/stats?${this._qs({ project, since })}`, undefined, { returnHeaders });
   }
 
   async getEvents(project, { event, days = 7, limit = 100 } = {}) {
-    return this.request('GET', `/events?${this._qs({ project, days, limit, event })}`);
+    const since = `${days}d`;
+    return this.request('GET', `/events?${this._qs({ project, since, limit, event })}`);
   }
 
   async getProperties(project, days = 30) {
-    return this.request('GET', `/properties?${this._qs({ project, days })}`);
+    const since = `${days}d`;
+    return this.request('GET', `/properties?${this._qs({ project, since })}`);
   }
 
   async getPropertiesReceived(project, { since, sample } = {}) {

test/api.test.mjs

@@ -139,7 +139,7 @@ describe('AgentAnalyticsAPI', () => {
 
     it('getStats → GET /stats with query params', async () => {
       await api.getStats('my-site', 30);
-      assert.equal(lastUrl, 'https://api.test/stats?project=my-site&days=30');
+      assert.equal(lastUrl, 'https://api.test/stats?project=my-site&since=30d');
     });
 
     it('getStats encodes project name', async () => {
@@ -149,17 +149,17 @@ describe('AgentAnalyticsAPI', () => {
 
     it('getEvents → GET /events with defaults', async () => {
       await api.getEvents('my-site');
-      assert.equal(lastUrl, 'https://api.test/events?project=my-site&days=7&limit=100');
+      assert.equal(lastUrl, 'https://api.test/events?project=my-site&since=7d&limit=100');
     });
 
     it('getEvents with event filter', async () => {
       await api.getEvents('my-site', { event: 'page_view', days: 14, limit: 50 });
-      assert.equal(lastUrl, 'https://api.test/events?project=my-site&days=14&limit=50&event=page_view');
+      assert.equal(lastUrl, 'https://api.test/events?project=my-site&since=14d&limit=50&event=page_view');
     });
 
     it('getProperties → GET /properties', async () => {
       await api.getProperties('my-site', 60);
-      assert.equal(lastUrl, 'https://api.test/properties?project=my-site&days=60');
+      assert.equal(lastUrl, 'https://api.test/properties?project=my-site&since=60d');
     });
   });
 

test/stats-render.test.mjs

@@ -0,0 +1,78 @@
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import { execFile } from 'node:child_process';
+import { createServer } from 'node:http';
+import { fileURLToPath } from 'node:url';
+import { dirname, join } from 'node:path';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const CLI = join(__dirname, '..', 'bin', 'cli.mjs');
+
+function startMockServer(responseData) {
+  return new Promise((resolve) => {
+    const server = createServer((req, res) => {
+      res.writeHead(200, { 'Content-Type': 'application/json' });
+      res.end(JSON.stringify(responseData));
+    });
+    server.listen(0, '127.0.0.1', () => {
+      const port = server.address().port;
+      resolve({ server, url: `http://127.0.0.1:${port}` });
+    });
+  });
+}
+
+function runCli(args, env) {
+  return new Promise((resolve) => {
+    execFile('node', [CLI, ...args], { timeout: 5000, env: { ...process.env, ...env } }, (err, stdout, stderr) => {
+      resolve({ code: err ? err.code : 0, stdout, stderr });
+    });
+  });
+}
+
+describe('stats command rendering', () => {
+  it('renders timeSeries as daily chart', async () => {
+    const { server, url } = await startMockServer({
+      totals: { total_events: 100, unique_users: 10 },
+      events: [{ event: 'page_view', count: 100, unique_users: 10 }],
+      timeSeries: [
+        { date: '2026-02-25', total_events: 40, unique_users: 5 },
+        { date: '2026-02-26', total_events: 60, unique_users: 8 },
+      ],
+    });
+
+    try {
+      const { stdout } = await runCli(['stats', 'test-project'], {
+        AGENT_ANALYTICS_API_KEY: 'aak_test',
+        AGENT_ANALYTICS_URL: url,
+      });
+
+      // Should render the daily chart section
+      assert.ok(stdout.includes('Daily:'), 'should show Daily heading');
+      assert.ok(stdout.includes('2026-02-25'), 'should show first date');
+      assert.ok(stdout.includes('2026-02-26'), 'should show second date');
+      assert.ok(stdout.includes('40 events'), 'should show first day count');
+      assert.ok(stdout.includes('60 events'), 'should show second day count');
+    } finally {
+      server.close();
+    }
+  });
+
+  it('does not crash when timeSeries is absent', async () => {
+    const { server, url } = await startMockServer({
+      totals: { total_events: 5, unique_users: 1 },
+      events: [],
+    });
+
+    try {
+      const { code, stdout } = await runCli(['stats', 'test-project'], {
+        AGENT_ANALYTICS_API_KEY: 'aak_test',
+        AGENT_ANALYTICS_URL: url,
+      });
+
+      assert.equal(code, 0);
+      assert.ok(!stdout.includes('Daily:'), 'should not show Daily when no timeSeries');
+    } finally {
+      server.close();
+    }
+  });
+});