@vike-io/cli 0.7.0 → 0.7.2

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes are documented here. Entries follow [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## 0.7.1 — eval harness v2 + help tweaks + SQL fix
+
+- **Eval schema upgraded** (`evals/questions.yaml`): multi-command list, per-skill tag, multi-step flag, word-boundary forbidden matching. Per-skill scoring breakdown + single-vs-multi-step split in the runner output.
+- **10 new multi-step questions** exercising composite playbook skills (smart-money-discovery, wallet-portfolio, polymarket-screener, perp-screener, hl-trader-profile, alerts-setup, etc.) — 32 questions total.
+- **Help discoverability tweaks**: `wallet counterparties --limit` and `wallet balances --min-usd` descriptions improved (eval surfaced both as commonly-missed flags).
+- **Backend bug fix shipped (server-side)**: `wallet_transactions` returned 0 rows due to ClickHouse SELECT-alias shadowing — fixed in monorepo, deployed to prod.
+
 ## 0.7.0 — 5 new commands (transactions, compare, trace, batch, screener)
 
 - **`vike wallet transactions <address>`** (alias `txs`) — raw per-wallet transfer list with `--direction in|out|all`, `--days`, `--min-usd`, `--token` filters.

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "AI-agent CLI for vike.io on-chain analytics, perps, options, and prediction markets across 11 chains",
   "type": "module",
   "main": "src/index.js",
@@ -65,11 +65,11 @@
     "node": ">=20.0.0"
   },
   "dependencies": {
-    "commander": "^12.1.0"
+    "commander": "^14.0.3"
   },
   "devDependencies": {
     "@changesets/cli": "^2.27.9",
-    "eslint": "^9.15.0",
-    "vitest": "^2.1.5"
+    "eslint": "^10.4.0",
+    "vitest": "^4.1.7"
   }
 }

package/src/commands/wallet.js

@@ -104,7 +104,7 @@ export function registerWallet(program) {
     .description('Top trading partners of a wallet by combined transfer volume')
     .option('--chain <c>', 'ethereum | bsc | base', 'ethereum')
     .option('--window <w>', '24h | 7d | 30d | 90d | 180d | all', '30d')
-    .option('--limit <n>', 'Max counterparties (max 50)', '10')
+    .option('--limit <n>', 'Max counterparties returned (default 10, max 50)', '10')
     .option('--json', 'Emit JSON')
     .action(async (address, opts) => {
       try {
@@ -139,7 +139,7 @@ export function registerWallet(program) {
     .command('balances <address>')
     .description('Current ERC-20 holdings across major EVM chains')
     .option('--chains <list>', 'Comma-separated chains (eth,bsc,base,arbitrum,polygon,optimism)')
-    .option('--min-usd <n>', 'Filter dust below this USD value', '1')
+    .option('--min-usd <n>', 'Filter dust below this USD value (default $1; recommend $100+ for noisy wallets)', '1')
     .option('--json', 'Emit JSON')
     .action(async (address, opts) => {
       try {

package/src/index.js

@@ -21,6 +21,7 @@ import {
   getUpgradeNotice,
   scheduleUpdateCheck,
 } from './update-check.js';
+import { trackCommandSucceeded, trackCommandFailed } from './telemetry.js';
 
 const version = readPackageVersion();
 
@@ -38,6 +39,19 @@ program
   .description('AI-agent CLI for vike.io on-chain analytics')
   .version(version, '-v, --version', 'Show CLI version');
 
+// Telemetry: time each command from preAction to postAction; failure path is
+// caught below via parseAsync().catch().
+const _cmdStart = new WeakMap();
+program.hook('preAction', (_thisCmd, actionCmd) => {
+  _cmdStart.set(actionCmd, Date.now());
+});
+program.hook('postAction', (_thisCmd, actionCmd) => {
+  const t0 = _cmdStart.get(actionCmd);
+  trackCommandSucceeded(actionCmd.name(), actionCmd.opts(), {
+    durationMs: t0 ? Date.now() - t0 : null,
+  });
+});
+
 registerLogin(program);
 registerLogout(program);
 registerInit(program);
@@ -52,10 +66,16 @@ registerPolymarket(program);
 registerLabels(program);
 registerWeb(program);
 
+const _parseT0 = Date.now();
 program.parseAsync(process.argv)
   .catch((err) => {
     console.error(`vike: ${err.message ?? err}`);
     process.exitCode = 1;
+    const cmd = program.args[0] || 'unknown';
+    trackCommandFailed(cmd, {}, {
+      errorCode: err?.code || err?.name || 'error',
+      durationMs: Date.now() - _parseT0,
+    });
   })
   .finally(() => {
     // Fire-and-forget background check, detached subprocess. Survives our exit.

package/src/telemetry.js

@@ -1,4 +1,4 @@
-// Anonymous, opt-out telemetry skeleton.
+// Anonymous, opt-out telemetry.
 //
 // Design decisions:
 // - Honors DO_NOT_TRACK=1 (industry standard) AND VIKE_NO_TELEMETRY=1
@@ -7,25 +7,24 @@
 // - Rolling 30-min session ID — same session_id within a 30-min window
 // - AbortController + 1000ms timeout, timer.unref() so we can't keep
 //   the process alive
-// - Payload includes: flag NAMES (no values), latency, success/error_code,
-//   chain (if relevant). NEVER the actual data the user queried.
+// - Payload includes: flag NAMES (no values), latency, success/error_code.
+//   NEVER the actual data the user queried (no addresses, no symbols).
 // - Fire-and-forget: errors are swallowed
-//
-// THE SINK IS NOT YET WIRED. `_send()` is a no-op until we have an
-// endpoint. Keeping the API shape stable now so we can flip the switch
-// later without a refactor.
 
-import { existsSync, mkdirSync, readFileSync, writeFileSync, statSync } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { join } from 'node:path';
 import { randomUUID } from 'node:crypto';
+import { readPackageVersion } from './util.js';
 
 const CONFIG_DIR = join(homedir(), '.vike');
 const ID_FILE = join(CONFIG_DIR, 'telemetry-id');
 const SESSION_FILE = join(CONFIG_DIR, 'session');
 const SESSION_TTL_MS = 30 * 60 * 1000;
-const ENDPOINT = process.env.VIKE_TELEMETRY_ENDPOINT || '';
+const ENDPOINT = process.env.VIKE_TELEMETRY_ENDPOINT
+  || 'https://vike.io/api/cli-telemetry';
 const REQUEST_TIMEOUT_MS = 1000;
+const CLI_VERSION = (() => { try { return readPackageVersion(); } catch { return ''; } })();
 
 function disabled() {
   return process.env.DO_NOT_TRACK === '1' ||
@@ -82,9 +81,13 @@ function _send(event, payload) {
   if (typeof timer.unref === 'function') timer.unref();
   fetch(ENDPOINT, {
     method: 'POST',
-    headers: { 'Content-Type': 'application/json' },
+    headers: {
+      'Content-Type': 'application/json',
+      'User-Agent': `vike-cli/${CLI_VERSION}`,
+    },
     body: JSON.stringify({
       event,
+      cli_version: CLI_VERSION,
       device_id: getDeviceId(),
       session_id: getSessionId(),
       ts: Date.now(),
@@ -98,8 +101,11 @@ function _send(event, payload) {
 
 function flagNames(opts) {
   // Extract flag names from a commander options object. Values stripped.
+  // Convert camelCase to kebab-case for readability in analytics.
   if (!opts || typeof opts !== 'object') return [];
-  return Object.keys(opts).filter((k) => !k.startsWith('_') && k !== 'parent');
+  return Object.keys(opts)
+    .filter((k) => !k.startsWith('_') && k !== 'parent')
+    .map((k) => '--' + k.replace(/[A-Z]/g, (m) => '-' + m.toLowerCase()));
 }
 
 export function trackCommandSucceeded(command, opts, { durationMs } = {}) {

package/src/update-check.js

@@ -11,8 +11,7 @@
 import { spawn } from 'node:child_process';
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { homedir } from 'node:os';
-import { dirname, join } from 'node:path';
-import { fileURLToPath } from 'node:url';
+import { join } from 'node:path';
 
 const CONFIG_DIR = join(homedir(), '.vike');
 const CHECK_FILE = join(CONFIG_DIR, 'update-check.json');
@@ -33,13 +32,6 @@ function readCache() {
   try { return JSON.parse(readFileSync(CHECK_FILE, 'utf8')); } catch { return null; }
 }
 
-function writeCache(latest) {
-  try {
-    if (!existsSync(CONFIG_DIR)) mkdirSync(CONFIG_DIR, { recursive: true, mode: 0o700 });
-    writeFileSync(CHECK_FILE, JSON.stringify({ latest, checkedAt: Date.now() }, null, 2));
-  } catch { /* ignore */ }
-}
-
 function cmpSemver(a, b) {
   const pa = a.split('.').map(Number);
   const pb = b.split('.').map(Number);
@@ -73,7 +65,6 @@ export function scheduleUpdateCheck() {
   const cache = readCache();
   if (cache && Date.now() - (cache.checkedAt || 0) < CHECK_INTERVAL_MS) return;
 
-  const __filename = fileURLToPath(import.meta.url);
   // Inline child script — no separate file needed, no dependency on
   // package layout, survives `npm install -g` symlinking.
   const code = `