alphacouncil-agent 0.5.5 → 0.6.0

package/.claude-plugin/marketplace.json

@@ -6,14 +6,14 @@
   },
   "metadata": {
     "description": "AlphaCouncil plugins for public-equity research.",
-    "version": "0.5.5"
+    "version": "0.6.0"
   },
   "plugins": [
     {
       "name": "alphacouncil-agent",
       "source": "./",
       "description": "Multi-agent equity research: evidence packets, bull/bear debate, portfolio-manager decision.",
-      "version": "0.5.5",
+      "version": "0.6.0",
       "author": {
         "name": "Zhao73"
       },

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "alphacouncil-agent",
-  "version": "0.5.5",
+  "version": "0.6.0",
   "description": "Multi-agent public-equity research workflow for Claude Code: spawns analyst workers, gathers sourced JSON evidence packets, runs bull/bear debate, and produces a portfolio-manager Buy/Overweight/Hold/Underweight/Sell decision with a full report.",
   "author": {
     "name": "Zhao73"

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "alphacouncil-agent",
-  "version": "0.5.5",
+  "version": "0.6.0",
   "description": "Host-visible or background public-equity research workflow with shared evidence packets.",
   "author": {
     "name": "Zhao73"

package/CHANGELOG.md

@@ -2,6 +2,37 @@
 
 Notable changes per release. Dates are UTC.
 
+## [0.6.0] — 2026-07-26
+
+Minor rather than patch: **every screened number changes.** Anyone comparing a result from
+0.5.x will see different figures, and the 0.5.x ones were wrong.
+
+### Fixed
+
+- **Quarterly and stub periods were counted as years.** A 10-K reports quarterly and
+  acquisition-stub periods alongside annual ones, and the reader keyed only on the end date.
+  Lumentum's fiscal 2015 became ten separate "years" — nine quarters and stubs plus the real
+  363-day period — so every multi-year rule averaged quarterly income against annual equity.
+  LITE's ten-year ROE read 2.26% and is **-1.64%**; MU's read 10.03% and is **12.83%**.
+  Periods must now span 300–400 days, wide enough for a 53-week fiscal year, with instant
+  balance-sheet facts exempt because they have no duration.
+- **Tag aliases stopped at the first match instead of merging.** Revenue moved to the ASC 606
+  tag in 2022, so a company filing since 2013 appeared to have four years of history — which
+  then fired a "listed under ten years" exemption on a decade-old filer. Aliases now merge,
+  the preferred one winning a contested year and a later filing winning within one alias.
+- **The dilution rule had never once been computed.** Share counts live under the XBRL
+  `shares` unit and the reader asked for `USD`, so it reported `skipped` for every company
+  ever screened — seven rules advertised, six ever computed, and a skip is indistinguishable
+  from a genuine data gap. AAPL now computes 7 of 7.
+
+### Testing
+
+- The existing dilution test passed throughout because its fixture also put share counts
+  under `USD` — the same mistake as the reader, so the two agreed with each other. The
+  fixture now uses the unit XBRL actually uses.
+- A regression test pins that one company's filings cannot leak into another's result.
+  Verified live as well: LITE, then AAPL, then LITE again, identical to the digit.
+
 ## [0.5.5] — 2026-07-26
 
 ### Fixed

package/mcp/lib/screen.mjs

@@ -17,8 +17,16 @@ const pct = (x) => (x === null ? null : Number((x * 100).toFixed(2)));
 const last = (series, n) => series.slice(-n);
 const sum = (xs) => xs.reduce((a, b) => a + b, 0);
 
+/**
+ * XBRL reports share counts under the `shares` unit, not `USD`. Requesting the default unit
+ * for a share concept returns nothing, so the dilution rule reported `skipped` for every
+ * company that has ever been screened -- seven rules advertised, six ever computed, and the
+ * skip looked exactly like a genuine data gap.
+ */
+const UNIT_BY_CONCEPT = { sharesOutstanding: "shares" };
+
 function values(facts, key, asOf) {
-  const found = annualSeries(facts, CONCEPTS[key], { asOf });
+  const found = annualSeries(facts, CONCEPTS[key], { asOf, unit: UNIT_BY_CONCEPT[key] || "USD" });
   return found ? found.series.map((e) => ({ end: e.end, filed: e.filed, val: e.val })) : [];
 }
 

package/mcp/lib/sec.mjs

@@ -70,26 +70,80 @@ export async function fetchCompanyFacts(cik) {
  * Tries several tags because the same economic quantity has different names depending on
  * when and under which taxonomy a company filed.
  */
+/**
+ * How long a reported period is, in days. Instant facts (balance-sheet items) have no start.
+ */
+const spanDays = (entry) =>
+  entry.start ? Math.round((Date.parse(entry.end) - Date.parse(entry.start)) / 86400000) : null;
+
+/**
+ * Is this entry an annual figure?
+ *
+ * A 10-K carries quarterly and stub periods alongside the annual ones. Keying only on the
+ * end date treated each as its own year: Lumentum's 2015 produced ten "years" from one
+ * fiscal year -- nine quarters and stubs plus the real 363-day period. Every multi-year rule
+ * then averaged quarterly income against annual equity, silently.
+ *
+ * Fiscal years run 52 or 53 weeks, so the window has to be wider than 365 exactly.
+ */
+const ANNUAL_MIN_DAYS = 300;
+const ANNUAL_MAX_DAYS = 400;
+const isAnnual = (entry) => {
+  const days = spanDays(entry);
+  // Instant facts have no duration: shares outstanding, equity, total assets.
+  if (days === null) return true;
+  return days >= ANNUAL_MIN_DAYS && days <= ANNUAL_MAX_DAYS;
+};
+
+/** The fiscal year a period belongs to, taken from its end date. */
+const fiscalYear = (entry) => Number(String(entry.end).slice(0, 4));
+
+/**
+ * Annual history for a concept, merged across every alias.
+ *
+ * Merging matters as much as the annual filter. The function used to return at the first
+ * alias with any data, and revenue moved to RevenueFromContractWithCustomerExcludingAssessedTax
+ * when ASC 606 was adopted -- so a company reporting under the new tag since 2022 looked like
+ * it had four years of history, which then fired a "listed under ten years" exemption on a
+ * company that had been public for a decade.
+ *
+ * Aliases are ordered by preference, so an earlier alias wins where both cover a year.
+ */
 export function annualSeries(facts, tags, { asOf = null, unit = "USD" } = {}) {
   const cutoff = asOf ? new Date(asOf).getTime() : null;
+  const byYear = new Map();
+  let usedTags = [];
+
   for (const tag of tags) {
     const entries = facts?.facts?.["us-gaap"]?.[tag]?.units?.[unit];
     if (!Array.isArray(entries) || entries.length === 0) continue;
+    let contributed = false;
 
-    const byPeriod = new Map();
     for (const entry of entries) {
       if (entry.form !== "10-K" || !entry.end || !Number.isFinite(entry.val)) continue;
+      if (!isAnnual(entry)) continue;
       // Look-ahead guard: a filing is only usable once it was actually filed.
       if (cutoff && new Date(entry.filed).getTime() > cutoff) continue;
-      const prior = byPeriod.get(entry.end);
-      // Keep the most recently filed value for a period: restatements supersede.
-      if (!prior || new Date(entry.filed) > new Date(prior.filed)) byPeriod.set(entry.end, entry);
+
+      const year = fiscalYear(entry);
+      const prior = byYear.get(year);
+      if (!prior) {
+        byYear.set(year, { ...entry, tag });
+        contributed = true;
+        continue;
+      }
+      // An earlier alias always wins the year; within one alias, the latest filing wins,
+      // because a restatement supersedes what it restates.
+      if (prior.tag === tag && new Date(entry.filed) > new Date(prior.filed)) {
+        byYear.set(year, { ...entry, tag });
+      }
     }
-    if (byPeriod.size === 0) continue;
-    const series = [...byPeriod.values()].sort((a, b) => new Date(a.end) - new Date(b.end));
-    return { tag, unit, series };
+    if (contributed) usedTags.push(tag);
   }
-  return null;
+
+  if (byYear.size === 0) return null;
+  const series = [...byYear.values()].sort((a, b) => new Date(a.end) - new Date(b.end));
+  return { tag: usedTags[0], tags: usedTags, unit, series };
 }
 
 /** Concept aliases, ordered by preference. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "alphacouncil-agent",
-  "version": "0.5.5",
+  "version": "0.6.0",
   "description": "Multi-agent public-equity research workflow plugin for Codex & Claude Code: sourced evidence packets, bull/bear debate, and a portfolio-manager Buy/Overweight/Hold/Underweight/Sell decision.",
   "type": "module",
   "license": "MIT",