test+fix: 34 e2e tests, fix intraday + market_cap_series, add agent frontmatter (0.4.2)

Added comprehensive test suite (stdlib-only, runs in CI when EODHD_API_TOKEN
secret is present):

- tests/test_python_client.py — 30 e2e cases hitting every endpoint
  in SUPPORTED_ENDPOINTS against live EODHD API. Each case validates
  exit code, JSON response, and reports shape. Subscription-gated
  endpoints classified as SKIP (not FAIL).

- tests/test_mcp_v1.py — full MCP protocol e2e: v2 OAuth challenge
  (must return 401), v1 initialize handshake (captures session ID),
  notifications/initialized, tools/list (verified 80 tools live),
  tools/call (invokes get_exchanges_list).

- tests/test_skill_references.py — static cross-reference validation:
  SKILL.md frontmatter, endpoint-doc links, eodhd_client.py example
  endpoints, slash-command skill refs, agent frontmatter, plugin.json
  capabilities.tools vs live MCP tool count.

Bugs surfaced and fixed:

- skills/eodhd-api/scripts/eodhd_client.py: --endpoint intraday now
  converts YYYY-MM-DD --from-date/--to-date to Unix timestamps (API
  rejects ISO date format for this endpoint with HTTP 422).

- skills/eodhd-api/scripts/market_cap_series.py: SharesOutstanding
  lives at SharesStats::* not Highlights::*. Also handles scalar
  return from "Field::Subfield" filter (was assuming dict).

- agents/financial-analyst.md: added YAML frontmatter (name +
  description) — same class of bug as the SKILL.md fix in PR #5.

- .claude-plugin/plugin.json + README.md: capabilities.tools 75 → 80
  to match actual MCP v1 tool count (verified via tools/list).

CI changes (.github/workflows/validate.yml):

- Static test (test_skill_references.py) runs on every push/PR.
- e2e tests run only if EODHD_API_TOKEN secret is configured.
- Forks / external PRs without secret access skip e2e (no false
  negatives).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: alexpipipi

.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "Developer-first financial data for Claude — 150k+ tickers, 70+ exchanges, MCP Server with OAuth, 7 curated workflow skills, Python client, 72 endpoint docs",
-    "version": "0.4.1"
+    "version": "0.4.2"
   },
   "plugins": [
     {

.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "eodhd-claude-skills",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Developer-first financial data for Claude — 150k+ tickers, 70+ exchanges, real-time & historical prices, fundamentals, options, technicals, news, sentiment, macro indicators, ESG, and more via EODHD API and MCP Server.",
   "author": {
     "name": "EOD Historical Data",
@@ -34,7 +34,7 @@
     }
   },
   "capabilities": {
-    "tools": 75,
+    "tools": 80,
     "tickers": "150,000+",
     "exchanges": "70+",
     "endpoints": 72,

.github/workflows/validate.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Validate plugin.json schema
         run: |
           python3 - <<'PY'
-          import json, sys
+          import json
           m = json.load(open('.claude-plugin/plugin.json'))
           assert isinstance(m.get('name'), str), 'name must be string'
           assert isinstance(m.get('version'), str), 'version must be string'
@@ -43,34 +43,12 @@ jobs:
           print(f'version sync OK: {p}')
           PY
 
-      - name: Validate SKILL.md frontmatter
-        run: |
-          pip install pyyaml --quiet
-          python3 - <<'PY'
-          import yaml, sys, pathlib
-          fails = []
-          for skill in pathlib.Path('skills').glob('*/SKILL.md'):
-              text = skill.read_text()
-              if not text.startswith('---'):
-                  fails.append(f'{skill}: missing YAML frontmatter')
-                  continue
-              fm_end = text.find('---', 3)
-              if fm_end == -1:
-                  fails.append(f'{skill}: unterminated frontmatter')
-                  continue
-              try:
-                  fm = yaml.safe_load(text[3:fm_end])
-              except Exception as e:
-                  fails.append(f'{skill}: YAML parse error: {e}')
-                  continue
-              for k in ('name', 'description'):
-                  if k not in fm:
-                      fails.append(f'{skill}: missing required key "{k}"')
-          if fails:
-              print('\n'.join(fails))
-              sys.exit(1)
-          print('all SKILL.md frontmatter OK')
-          PY
+      - name: Static skill/agent/command references
+        run: python3 tests/test_skill_references.py
+        env:
+          # If repo has an EODHD_API_TOKEN secret, the manifest tool-count check
+          # also cross-validates against live MCP server. Missing secret = check skipped.
+          EODHD_API_TOKEN: ${{ secrets.EODHD_API_TOKEN }}
 
   mcp-endpoint:
     runs-on: ubuntu-latest
@@ -95,3 +73,34 @@ jobs:
             000)         echo "FAIL: connection error"; exit 1 ;;
             *)           echo "WARN: unexpected $CODE — review manually"; exit 0 ;;
           esac
+
+  e2e-python-client:
+    # Run only when EODHD_API_TOKEN secret is configured in repo settings.
+    # On forks / un-secret PRs, this job is skipped (no false negatives).
+    runs-on: ubuntu-latest
+    if: ${{ github.event_name == 'push' || github.event.pull_request.head.repo.full_name == github.repository }}
+    steps:
+      - uses: actions/checkout@v4
+      - name: Check token availability
+        id: token
+        env:
+          EODHD_API_TOKEN: ${{ secrets.EODHD_API_TOKEN }}
+        run: |
+          if [ -n "$EODHD_API_TOKEN" ]; then
+            echo "have=true" >> "$GITHUB_OUTPUT"
+          else
+            echo "have=false" >> "$GITHUB_OUTPUT"
+            echo "::warning::EODHD_API_TOKEN secret not set — e2e tests skipped"
+          fi
+
+      - name: Run Python client e2e against live API
+        if: steps.token.outputs.have == 'true'
+        env:
+          EODHD_API_TOKEN: ${{ secrets.EODHD_API_TOKEN }}
+        run: python3 tests/test_python_client.py
+
+      - name: Run MCP v1 + v2 e2e
+        if: steps.token.outputs.have == 'true'
+        env:
+          EODHD_API_TOKEN: ${{ secrets.EODHD_API_TOKEN }}
+        run: python3 tests/test_mcp_v1.py

README.md

@@ -6,7 +6,7 @@ Developer-first financial data for Claude — 150,000+ tickers across 70+ exchan
 
 ## Highlights
 
-- **MCP Server v2** with OAuth 2.0 — 75 tools, 100+ embedded docs, smart ticker resolution
+- **MCP Server v2** with OAuth 2.0 — 80 tools, 100+ embedded docs, smart ticker resolution
 - **7 curated workflow skills** — company briefs, earnings, market overview, screening, risk, macro, options
 - **Financial analyst agent** — autonomous multi-source analysis
 - **5 slash commands** — `/eodhd-analyze`, `/eodhd-compare`, `/eodhd-market`, `/eodhd-screen`, `/eodhd-macro`
@@ -168,7 +168,7 @@ The `agents/financial-analyst.md` agent autonomously:
 ## MCP Server
 
 The EODHD MCP Server v2 provides:
-- **75 tools** across 15 categories (prices, fundamentals, options, technicals, news, sentiment, macro, etc.)
+- **80 tools** across 15 categories (prices, fundamentals, options, technicals, news, sentiment, macro, etc.)
 - **OAuth 2.0** authentication
 - **100+ embedded docs** as MCP resources
 - **3 prompt templates** for common workflows

agents/financial-analyst.md

@@ -1,3 +1,14 @@
+---
+name: financial-analyst
+description: >-
+  Autonomous financial analyst powered by EODHD market data. Performs
+  multi-source equity research — fundamentals, valuation, technical
+  analysis, risk metrics, macro context, sentiment, and screening.
+  Use when the user asks for an investment thesis, in-depth company
+  analysis, portfolio risk review, or any task that benefits from
+  combining several EODHD endpoints into a coherent report.
+---
+
 # Agent: Financial Analyst
 
 ## Identity

skills/eodhd-api/scripts/eodhd_client.py

@@ -85,6 +85,7 @@
 from __future__ import annotations
 
 import argparse
+import datetime
 import json
 import os
 import sys
@@ -382,6 +383,17 @@ def main() -> int:
     if args.interval:
         params["interval"] = args.interval
 
+    # Intraday endpoint requires Unix timestamps for from/to (not YYYY-MM-DD)
+    if args.endpoint == "intraday":
+        for key in ("from", "to"):
+            value = params.get(key)
+            if isinstance(value, str) and "-" in value:
+                try:
+                    dt = datetime.datetime.strptime(value, "%Y-%m-%d")
+                    params[key] = int(dt.replace(tzinfo=datetime.timezone.utc).timestamp())
+                except ValueError:
+                    pass  # Leave unchanged; API will surface the error
+
     # Technical indicators
     if args.function:
         params["function"] = args.function

skills/eodhd-api/scripts/market_cap_series.py

@@ -6,7 +6,7 @@
 
 Data sources (EODHD API):
   - /eod/{SYMBOL}          → daily OHLCV (close price)
-  - /fundamentals/{SYMBOL} → SharesOutstanding from Highlights
+  - /fundamentals/{SYMBOL} → SharesOutstanding from SharesStats
 
 For US stocks only, an alternative --method=api flag uses the dedicated
 /historical-market-cap/{SYMBOL} endpoint (weekly frequency, from 2019).
@@ -67,10 +67,13 @@ def get_shares_outstanding(symbol: str, token: str) -> float | None:
     params = urllib.parse.urlencode({
         "api_token": token,
         "fmt": "json",
-        "filter": "Highlights::SharesOutstanding",
+        "filter": "SharesStats::SharesOutstanding",
     })
     url = f"{BASE_URL}/fundamentals/{symbol}?{params}"
     data = fetch_json(url)
+    # A "Field::Subfield" filter returns the scalar value directly, not a dict
+    if isinstance(data, (int, float)):
+        return float(data)
     if isinstance(data, dict):
         if "error" in data:
             raise RuntimeError(f"Fundamentals API error: {data['error']}")