agent-notes 2.25.0__tar.gz → 2.27.0__tar.gz

{agent_notes-2.25.0 → agent_notes-2.27.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-notes
-Version: 2.25.0
+Version: 2.27.0
 Summary: AI agent configuration manager for Claude Code, OpenCode, and Copilot
 Author-email: Eugene Naumov <min.verkligheten@gmail.com>
 License-Expression: MIT

agent_notes-2.27.0/agent_notes/VERSION

@@ -0,0 +1 @@
+2.27.0

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/commands/doctor.py

@@ -2,6 +2,8 @@
 
 from pathlib import Path
 
+from ..registries.skill_registry import load_skill_registry
+
 # Re-export for backward compatibility. New code should import from agent_notes.domain.
 from ..domain.diagnostics import Issue, FixAction  # noqa: F401
 
@@ -42,6 +44,42 @@ from ..services.fs import (
     files_differ
 )
 
+# Canonical group vocabulary for skill frontmatter.
+# "process" and "domain" come from the base skills in data/skills/.
+# "rails", "docker", and "kamal" come from the sub-skills that are generated
+# during the release/packaging step and land in data/skills/ of the built
+# package (e.g. rails-models → group: rails, rails-kamal → group: kamal,
+# docker-compose → group: docker).  There is no single source-of-truth
+# constant elsewhere in the codebase, so the full vocabulary is listed here.
+_VALID_GROUPS = {"process", "domain", "rails", "docker", "kamal"}
+_VALID_MEMORY_BACKENDS = {"obsidian", "wiki", "local", "none"}
+
+
+def check_skill_frontmatter(scope: str, issues: list, fix_actions: list, profile_label: str = "") -> None:
+    """Warn (non-fatal) about skill frontmatter violations.
+
+    Checks every skill for:
+    - non-empty name and description
+    - group, if present, is in {"process", "domain"}
+    - requires_memory tokens, if present, are each in {"obsidian", "wiki", "local", "none"}
+
+    Violations are printed as advisories and do NOT affect issues/fix_actions or exit code.
+    """
+    registry = load_skill_registry()
+    for skill in registry.all():
+        if not skill.name:
+            print(f"  [skill-frontmatter] {skill.path.name}: 'name' is empty")
+        if not skill.description:
+            print(f"  [skill-frontmatter] {skill.path.name}: 'description' is empty")
+        if skill.group and skill.group not in _VALID_GROUPS:
+            print(f"  [skill-frontmatter] {skill.name}: 'group' value '{skill.group}' is not in {sorted(_VALID_GROUPS)}")
+        if skill.requires_memory:
+            for token in skill.requires_memory.split(","):
+                token = token.strip()
+                if token and token not in _VALID_MEMORY_BACKENDS:
+                    print(f"  [skill-frontmatter] {skill.name}: 'requires_memory' token '{token}' is not in {sorted(_VALID_MEMORY_BACKENDS)}")
+
+
 def _check_session_hook(scope: str, issues: list) -> None:
     """Check that the Claude Code SessionStart hook is registered in settings.json."""
     from ..services.settings_writer import has_hook
@@ -162,6 +200,9 @@ def diagnose(scope: str, fix: bool = False) -> bool:
     # SessionStart hook check (Claude Code only)
     _check_session_hook(scope, issues)
 
+    # Skill frontmatter advisory check (warn-only, non-fatal)
+    check_skill_frontmatter(scope, issues, fix_actions)
+
     # Print role→model assignments
     state = load_current_state()
     if state is not None:

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/commands/memory/notes.py

@@ -45,8 +45,10 @@ def do_add(title: str, body: str, note_type: str = "context", agent: str = "", p
         )
         print(f"{Color.GREEN}Note saved: {note_path}{Color.NC}")
     else:
-        print("The `add` subcommand is for obsidian or wiki storage.")
-        print("For local storage, write files directly to the agent subdirectory.")
+        import sys
+        print("The `add` subcommand is for obsidian or wiki storage.", file=sys.stderr)
+        print("For local storage, write files directly to the agent subdirectory.", file=sys.stderr)
+        sys.exit(1)
 
 
 def do_list() -> None:

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/agents/shared/cost_reporting.md

@@ -4,10 +4,14 @@ At the END of every response, run `agent-notes cost-report` and include the outp
 
 **Session cost** (cumulative for the entire conversation):
 
-Render every column the `agent-notes cost-report` CLI emits — `agent(model)`, `in/out/cache`, `time`, `actual`, `vs Claude Opus 4.7` — in that order. Do not split, drop, or rename columns. Preserve the data verbatim.
+Render every column the `agent-notes cost-report` CLI emits — `agent(model)`, `in/out/cache`, `time`, `actual`, `vs Claude Opus 4.8` — in that order. Do not split, drop, or rename columns. Preserve the data verbatim.
 
 **On failure or skip — never fabricate.** If `agent-notes cost-report` returns non-zero, errors, or you skip running it, do NOT render a placeholder table or invent rows like `(cost report unavailable — agent-notes cost-report not run)`. Instead, print one plain line under the heading:
 
 `Cost report skipped: <one-line reason>`
 
 If the command ran but produced an error message, print the error verbatim under the heading instead of a table. Fabricating a table when the CLI did not run is a violation.
+
+Cost table: once at end of response, never mid-response.
+
+Always include the cost report at the end of every response.

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/agents/shared/execution.md

@@ -86,7 +86,7 @@ Never spawn one agent per bullet point from the user's prompt. Combine related s
 
 Skip agents for: simple questions (answer directly), single-file edits (coder alone), or two-grep lookups (do it yourself).
 
-Give each agent a specific task with all context (paths, criteria). Always include the cost report at the end of every response.
+Give each agent a specific task with all context (paths, criteria).
 
 ### Permission pre-check (HARD RULE)
 

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/agents/shared/guardrails.md

@@ -9,10 +9,8 @@
 7. "Let me just verify this one thing" followed by 10 reads → if verification needs 10 reads, dispatch.
 8. Breaking tasks into steps so small they have no independent value → group into meaningful chunks.
 9. Writing a plan that only restates the user's words → a plan must include discovery findings, dependency order, and flagged risks.
-10. Skipping the cost report at the end of a response → always include it.
-11. Fabricating a cost-report table or placeholder rows when `agent-notes cost-report` did not run successfully → forbidden. Print "Cost report skipped: <reason>" on a single line instead.
-12. Reporting "done" before tests pass and plan items match → forbidden by Done Gate.
-13. Reporting "done" / "complete" / "shipped" without an `agent-notes memory add ... session lead` call covering this work → forbidden by the Done Gate.
+10. Reporting "done" before tests pass and plan items match → forbidden by Done Gate.
+11. Reporting "done" / "complete" / "shipped" without an `agent-notes memory add ... session lead` call covering this work → forbidden by the Done Gate.
 
 ## Done Gate (HARD RULE)
 

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/agents/shared/hard_limits.md

@@ -13,7 +13,7 @@ You MUST NOT directly:
 - Run installs, builds, migrations, or destructive commands — dispatch `devops`
 - Use `bash` for anything beyond the read-only verification list above
 
-If you feel the urge to "just quickly check a file" — STOP. Dispatch `explorer`. Every file read by the lead is a budget leak (Opus tokens are 30× Haiku).
+If you feel the urge to "just quickly check a file" — STOP. Dispatch `explorer`. Every file read by the lead is a budget leak (Opus tokens are 5× Haiku).
 
 Exception: trivial requests (factual questions, conversational replies, single-line answers) may be handled inline with no tools.
 
@@ -25,4 +25,3 @@ Exception: trivial requests (factual questions, conversational replies, single-l
 - Plans: structured bullet lists with file paths. No prose paragraphs.
 - Agent briefings: context + task + acceptance criteria. No commentary or justification.
 - Never narrate internal deliberation — report outcomes only.
-- Cost table: once at end of response, never mid-response.

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/pricing.yaml

@@ -1,5 +1,5 @@
 baseline:
-  label: Claude Opus 4.7
+  label: Claude Opus 4.8
   price:
     in: 5.00
     out: 25.00
@@ -16,8 +16,8 @@ providers:
         match: "*sonnet*"
         price: {in: 3.00, out: 15.00, cache: 0.30}
         updated_at: "2026-04"
-      - name: Claude Opus 4.7 / 4.6 / 4.5
-        match: ["*opus-4.7*", "*opus-4.6*", "*opus-4.5*"]
+      - name: Claude Opus 4.8 / 4.7 / 4.6 / 4.5
+        match: ["*opus-4.8*", "*opus-4.7*", "*opus-4.6*", "*opus-4.5*"]
         price: {in: 5.00, out: 25.00, cache: 0.50}
         updated_at: "2026-04"
       - name: Claude Opus (legacy)

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/skills/code-review/SKILL.md

@@ -47,14 +47,16 @@ Work through these five lenses in order. Report findings grouped by lens, ranked
 
 ## Output format
 
+Emit each finding as a record: `severity` (blocking | suggestion) · `file` · `line` · `finding` · `why` · `fix` (if applicable). When the output is consumed by another agent, emit JSON objects with exactly those keys. For a human summary, group by severity:
+
 ```
 BLOCKING
-- [file:line] [finding] — [why it matters]
+- [file:line] [finding] — [why it matters] → [fix]
 
 SUGGESTIONS
 - [file:line] [finding] — [alternative if applicable]
 
-APPROVED (if no blocking issues)
+APPROVED (state explicitly when there are no blocking findings)
 ```
 
 A BLOCKING finding must be resolved before merge. A SUGGESTION is optional.

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/skills/improve-codebase-architecture/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: improve-codebase-architecture
-description: "Find deepening opportunities — modules where the interface is nearly as complex as the implementation. Use when user wants to improve architecture, reduce coupling, make code more testable, or says 'clean up the design'."
+description: "Find deepening opportunities — modules where the interface is nearly as complex as the implementation. Use when user wants to improve architecture, reduce coupling, make code more testable, or says 'clean up the design'. For broad code-quality hardening use rsi; for a single safe refactor use refactoring-protocol."
 group: process
 ---
 

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/skills/migrate-memory/SKILL.md

@@ -2,7 +2,7 @@
 name: migrate-memory
 description: "Reconcile the Obsidian vault or Wiki knowledge base to the latest canonical format. Use after upgrading agent-notes or when vault structure looks outdated or inconsistent."
 group: process
-requires_memory: obsidian, wiki
+requires_memory: obsidian,wiki
 ---
 
 # Migrate Memory

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/data/skills/refactoring-protocol/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: refactoring-protocol
-description: "Safe refactoring: green tests first, one extraction at a time, structure OR behavior never both. Use when user wants to refactor code, reduce duplication, or says 'clean this up'."
+description: "Safe refactoring: green tests first, one extraction at a time, structure OR behavior never both. Use when user wants to refactor code, reduce duplication, or says 'clean this up'. For an iterative multi-dimensional cleanup loop use rsi; for module or interface redesign use improve-codebase-architecture."
 group: process
 ---
 

agent_notes-2.27.0/agent_notes/data/skills/rsi/SKILL.md

@@ -0,0 +1,60 @@
+---
+name: rsi
+description: "Recursive code-improvement loop: iteratively harden an existing codebase across bugs, performance, quality, consistency/homogeneity, pattern adherence, tests, atomicity, independence, and DRY — strictly without adding functionality. Use when the user wants to improve, clean up, or harden existing code, says 'rsi', or asks to raise code quality without new features. For a single one-shot refactor use refactoring-protocol; for module or interface redesign use improve-codebase-architecture."
+group: process
+argument-hint: "[path or scope, optional]"
+---
+
+# RSI — Recursive Self-Improvement (Code)
+
+Iteratively improve EXISTING code until it stops yielding improvements. The lead orchestrates; specialized agents do the work. This is a quality loop, not a feature loop.
+
+## Prime invariant
+
+- **Behavior-preserving.** NEVER add functionality, new features, new public API, or new config. Only improve what already exists.
+- **Tests green before a pass and after every change.** If the target has no test coverage, write a characterization test FIRST, then improve.
+- **One change = one concern.** Structure OR behavior, never both in the same change.
+
+## The loop (lead-orchestrated, loop-until-dry)
+
+1. **Scope.** Resolve the target (arg path, or whole project). Identify the test command and confirm the suite is green. If it is red, stop and report — fix the suite before improving.
+2. **Scan — one dimension at a time.** Dispatch read-only agents to produce a ranked list of concrete opportunities:
+   - `debugger` / `security-auditor` → bugs, correctness, vulnerabilities
+   - `performance-profiler` → hot paths, N+1, redundant work
+   - `system-auditor` → duplication, dead code, coupling, inconsistent implementations of one concept
+   - `reviewer` → readability, naming, pattern & consistency adherence
+   - `test-writer` (read-only pass) → coverage gaps
+
+   Each opportunity is a **finding record**: `file` · `line` · `dimension` · `severity` (blocker | major | minor) · `why` · `fix` (one-line proposed change). When a downstream agent consumes the scan, emit findings as JSON objects with exactly those keys; otherwise the dashed form `file:line — dimension/severity — why — fix` is fine.
+
+3. **Prioritize.** Order: correctness/safety > missing tests on touched code > DRY/duplication > consistency/homogeneity > pattern & convention fit > performance > clarity/naming. Drop anything that changes behavior or adds capability.
+4. **Apply ONE atomic, independent change.** Dispatch `coder` (bugfix) or `refactorer` (behavior-preserving cleanup). Smallest viable diff.
+5. **Verify.** Run affected tests — must stay green. `reviewer` confirms: no behavior change, fits conventions, genuinely improves the dimension. On regression → revert and re-plan.
+6. **Commit (auto, atomic).** One concern per commit, independent and revertable. Use the `git` skill's message format. Then take the next opportunity.
+7. **Repeat passes.** Converge when TWO consecutive full passes surface no new actionable improvement. Then report.
+
+## Dimensions rubric
+
+Each dimension: what to **hunt**, what to **fix**, what to **leave alone**.
+
+- **Bugs & correctness** — off-by-one, nil/None, races, missing error handling, unhandled edge cases. Fix minimally + add a regression test. Don't redesign.
+- **Performance** — measured hot paths, N+1 queries, redundant work, bad complexity, unbounded growth. Don't micro-optimize cold paths or trade clarity for guesswork.
+- **Code quality** — long methods, deep nesting, unclear names, magic values. Extract, apply guard clauses, rename. Don't gold-plate.
+- **Consistency / homogeneity** — the same kind of thing implemented multiple different ways: mixed styles for one object type, divergent shapes for one concept, inconsistent signatures or return types. Converge on one canonical form and reduce needless variation. Distinct from DRY — this targets divergent *expression* of one concept, not duplicated *logic*. Don't force genuinely different things into a false-common shape.
+- **Pattern adherence** — match the project's dominant idioms, structure, and conventions. Align outliers. Don't invent new patterns.
+- **Tests** — cover changed or risky code, fix flaky/slow tests, add missing edge cases. Don't test trivial getters.
+- **Atomicity** — split god-functions and god-classes into single-responsibility units. Don't over-fragment.
+- **Independence** — reduce coupling, remove hidden global state, narrow interfaces. Don't add abstraction layers nobody needs.
+- **DRY** — collapse genuine duplication into one source of truth. Don't DRY accidental similarity (premature abstraction is worse than duplication).
+- **Dead code** — unused functions, variables, imports, branches. Delete after confirming no external use.
+
+## Gates (hard)
+
+- **Green-before / green-after.** A red suite halts the loop.
+- **No-feature gate.** If a change adds capability, it is out of scope — reject it.
+- **Atomic & independent commits.** Revert on regression with `git revert` — never `reset --hard` or force-push.
+- **Max 2 review rounds per change,** then accept or drop it.
+
+## Done
+
+Report: improvements grouped by dimension, commits made, tests added, deferrals (with reasons), and anything skipped as out-of-scope (new functionality).

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/registries/skill_registry.py

@@ -86,7 +86,8 @@ def _parse_skill_frontmatter(skill_md_path: Path) -> tuple[str, Optional[str], O
                     elif key == 'description':
                         description = value
                     elif key == 'requires_memory':
-                        requires_memory = value
+                        tokens = [t.strip() for t in value.split(",")]
+                        requires_memory = ",".join(t for t in tokens if t)
 
             # If no description in frontmatter, use first non-empty line after frontmatter
             if not description:

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/scripts/_claude_backend.py

@@ -6,8 +6,7 @@ from pathlib import Path
 
 from . import _pricing
 from ._formatting import (
-    BOLD, DIM, GREEN, YELLOW, NC,
-    tier_color, fmt_tokens, fmt_cost, fmt_time,
+    fmt_time, render_cost_table,
 )
 from ..services.state_store import state_file as _state_file, load_state as _load_state
 
@@ -279,66 +278,15 @@ def run(since: float | None = None, session_id: str | None = None) -> int:
             ms = agent_time_ms.get(label, 0)
         return fmt_time(ms / 1000) if ms > 0 else "n/a"
 
-    agent_col_w = max(len(f"{a}({m})") for a, m, *_ in costs) + 2
-    tok_col_w = max(
-        max(len(fmt_tokens(i, o, c)) for _, _, i, o, c, *_ in costs),
-        len(fmt_tokens(
-            sum(i for _, _, i, *_ in costs),
-            sum(o for _, _, _, o, *_ in costs),
-            sum(c for _, _, _, _, c, *_ in costs),
-        ))
-    ) + 2
-    all_time_strs = [_time_str_for(a) for a, *_ in costs]
-    time_col_w = max(len(s) for s in all_time_strs) + 2
-    W = (agent_col_w, tok_col_w, time_col_w, 12, 12)
-
-    bl_label = _pricing.baseline_label()
-    header = (
-        f"{'agent(model)':<{W[0]}}"
-        f" {'in/out/cache':<{W[1]}}"
-        f" {'time':<{W[2]}}"
-        f" {'actual':<{W[3]}}"
-        f" {f'vs {bl_label}':<{W[4]}}"
+    total_time_ms = sum(
+        lead_time_ms if agent == "lead" else agent_time_ms.get(agent, 0)
+        for agent, *_ in costs
     )
-    print(BOLD + header + NC)
-    print(DIM + "-" * len(header) + NC)
-
-    total_inp = total_outp = total_cache = 0
-    total_actual = total_vs = 0.0
-    total_time_ms = 0
-
-    for agent, model, inp, outp, cache, actual, vs in costs:
-        label = f"{agent}({model})"
-        col = tier_color(model)
-        t_str = _time_str_for(agent)
-        print(
-            col + f"{label:<{W[0]}}" + NC
-            + f" {fmt_tokens(inp, outp, cache):<{W[1]}}"
-            + f" {t_str:<{W[2]}}"
-            + f" {fmt_cost(actual):<{W[3]}}"
-            + f" {fmt_cost(vs):<{W[4]}}"
-        )
-        total_inp += inp
-        total_outp += outp
-        total_cache += cache
-        total_actual += actual
-        total_vs += vs
-        if agent == "lead":
-            total_time_ms += lead_time_ms
-        else:
-            total_time_ms += agent_time_ms.get(agent, 0)
-
-    saved_pct = round((1 - total_actual / total_vs) * 100) if total_vs else 0
-    total_label = f"TOTAL (saved {saved_pct}%)"
     total_time_str = fmt_time(total_time_ms / 1000) if total_time_ms > 0 else "n/a"
-    col = GREEN if total_actual <= 5 else YELLOW
-    print(
-        col + BOLD
-        + f"{total_label:<{W[0]}}"
-        + f" {fmt_tokens(total_inp, total_outp, total_cache):<{W[1]}}"
-        + f" {total_time_str:<{W[2]}}"
-        + f" {fmt_cost(total_actual):<{W[3]}}"
-        + f" {fmt_cost(total_vs):<{W[4]}}"
-        + NC
-    )
+
+    rows = [
+        (f"{agent}({model})", model, inp, outp, cache, _time_str_for(agent), actual, vs)
+        for agent, model, inp, outp, cache, actual, vs in costs
+    ]
+    render_cost_table(rows, total_time_str, _pricing.baseline_label())
     return 0

agent_notes-2.27.0/agent_notes/scripts/_formatting.py

@@ -0,0 +1,120 @@
+"""ANSI color constants and shared formatting helpers."""
+import os
+import sys
+
+_USE_COLOR = sys.stdout.isatty() and not os.environ.get("NO_COLOR")
+
+BOLD   = "\033[1m"   if _USE_COLOR else ""
+DIM    = "\033[2m"   if _USE_COLOR else ""
+YELLOW = "\033[0;33m" if _USE_COLOR else ""
+GREEN  = "\033[0;32m" if _USE_COLOR else ""
+CYAN   = "\033[0;36m" if _USE_COLOR else ""
+NC     = "\033[0m"   if _USE_COLOR else ""
+
+
+def tier_color(model_id: str) -> str:
+    if not _USE_COLOR:
+        return ""
+    if "opus" in model_id:
+        return YELLOW
+    if "sonnet" in model_id:
+        return CYAN
+    return DIM
+
+
+def fmt_num(n: int) -> str:
+    if n >= 1_000_000:
+        return f"{n / 1_000_000:.2f}m"
+    if n >= 1_000:
+        return f"{n / 1_000:.2f}k"
+    return str(n)
+
+
+def fmt_tokens(inp, outp, cache) -> str:
+    return f"{fmt_num(inp)}/{fmt_num(outp)}/{fmt_num(cache)}"
+
+
+def fmt_time(sec: float) -> str:
+    s = int(round(sec))
+    if s < 60:
+        return f"{s}s"
+    m, s = divmod(s, 60)
+    if m < 60:
+        return f"{m}m {s}s" if s else f"{m}m"
+    h, m = divmod(m, 60)
+    return f"{h}h {m}m" if m else f"{h}h"
+
+
+def fmt_cost(c: float) -> str:
+    return f"${c:.4f}"
+
+
+def render_cost_table(
+    rows: list[tuple],
+    total_time_str: str,
+    baseline_label: str,
+) -> None:
+    """Print a cost table to stdout.
+
+    Each row must be a tuple of:
+        (label: str, model: str, inp: int, outp: int, cache: int,
+         time_str: str, actual: float, vs: float)
+
+    total_time_str  — pre-formatted total time (backends compute it differently)
+    baseline_label  — label for the rightmost "vs" column header
+    """
+    agent_col_w = max(len(label) for label, *_ in rows) + 2
+    tok_col_w = max(
+        max(len(fmt_tokens(i, o, c)) for _, _, i, o, c, *_ in rows),
+        len(fmt_tokens(
+            sum(i for _, _, i, *_ in rows),
+            sum(o for _, _, _, o, *_ in rows),
+            sum(c for _, _, _, _, c, *_ in rows),
+        ))
+    ) + 2
+    time_col_w = max(
+        max(len(t) for _, _, _, _, _, t, *_ in rows),
+        len(total_time_str)
+    ) + 2
+    W = (agent_col_w, tok_col_w, time_col_w, 12, 12)
+
+    header = (
+        f"{'agent(model)':<{W[0]}}"
+        f" {'in/out/cache':<{W[1]}}"
+        f" {'time':<{W[2]}}"
+        f" {'actual':<{W[3]}}"
+        f" {f'vs {baseline_label}':<{W[4]}}"
+    )
+    print(BOLD + header + NC)
+    print(DIM + "-" * len(header) + NC)
+
+    total_inp = total_outp = total_cache = 0
+    total_actual = total_vs = 0.0
+
+    for label, model, inp, outp, cache, time_str, actual, vs in rows:
+        col = tier_color(model)
+        print(
+            col + f"{label:<{W[0]}}" + NC
+            + f" {fmt_tokens(inp, outp, cache):<{W[1]}}"
+            + f" {time_str:<{W[2]}}"
+            + f" {fmt_cost(actual):<{W[3]}}"
+            + f" {fmt_cost(vs):<{W[4]}}"
+        )
+        total_inp += inp
+        total_outp += outp
+        total_cache += cache
+        total_actual += actual
+        total_vs += vs
+
+    saved_pct = round((1 - total_actual / total_vs) * 100) if total_vs else 0
+    total_label = f"TOTAL (saved {saved_pct}%)"
+    col = GREEN if total_actual <= 5 else YELLOW
+    print(
+        col + BOLD
+        + f"{total_label:<{W[0]}}"
+        + f" {fmt_tokens(total_inp, total_outp, total_cache):<{W[1]}}"
+        + f" {total_time_str:<{W[2]}}"
+        + f" {fmt_cost(total_actual):<{W[3]}}"
+        + f" {fmt_cost(total_vs):<{W[4]}}"
+        + NC
+    )

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/scripts/_opencode_backend.py

@@ -3,10 +3,7 @@ import sqlite3
 from pathlib import Path
 
 from . import _pricing
-from ._formatting import (
-    BOLD, DIM, GREEN, YELLOW, NC,
-    tier_color, fmt_tokens, fmt_time, fmt_cost,
-)
+from ._formatting import fmt_time, render_cost_table
 
 DB = Path.home() / ".local/share/opencode/opencode.db"
 
@@ -63,7 +60,7 @@ def run() -> int:
         return 0
 
     records = [
-        (agent, model or "unknown", inp or 0, outp or 0, cache or 0, sec or 0)
+        (agent, _pricing.normalize_model(model or "unknown"), inp or 0, outp or 0, cache or 0, sec or 0)
         for agent, model, inp, outp, cache, sec in rows
     ]
 
@@ -74,68 +71,13 @@ def run() -> int:
         for agent, model, inp, outp, cache, sec in records
     ]
 
-    _total_inp  = sum(i for _, _, i, *_ in costs)
-    _total_outp = sum(o for _, _, _, o, *_ in costs)
-    _total_cache = sum(c for _, _, _, _, c, *_ in costs)
-    _max_sec    = max(s for _, _, _, _, _, s, *_ in costs)
-    _total_sec  = sum(s for _, _, _, _, _, s, *_ in costs)
-    _total_time = f"{fmt_time(_max_sec)} / {fmt_time(_total_sec)} seq"
-
-    agent_col_w = max(len(f"{a}({m})") for a, m, *_ in costs) + 2
-    tok_col_w = max(
-        max(len(fmt_tokens(i, o, c)) for _, _, i, o, c, *_ in costs),
-        len(fmt_tokens(_total_inp, _total_outp, _total_cache))
-    ) + 2
-    time_col_w = max(
-        max(len(fmt_time(s)) for _, _, _, _, _, s, *_ in costs),
-        len(_total_time)
-    ) + 2
-    W = (agent_col_w, tok_col_w, time_col_w, 12, 12)
-
-    bl_label = _pricing.baseline_label()
-    header = (
-        f"{'agent(model)':<{W[0]}}"
-        f" {'in/out/cache':<{W[1]}}"
-        f" {'time':<{W[2]}}"
-        f" {'actual':<{W[3]}}"
-        f" {f'vs {bl_label}':<{W[4]}}"
-    )
-    print(BOLD + header + NC)
-    print(DIM + "-" * len(header) + NC)
+    max_sec = max(s for _, _, _, _, _, s, *_ in costs)
+    total_sec = sum(s for _, _, _, _, _, s, *_ in costs)
+    total_time_str = f"{fmt_time(max_sec)} / {fmt_time(total_sec)} seq"
 
-    total_inp = total_outp = total_cache = 0
-    total_actual = total_vs = max_sec = total_sec = 0.0
-
-    for agent, model, inp, outp, cache, sec, actual, vs in costs:
-        label = f"{agent}({model})"
-        time_str = fmt_time(sec)
-        col = tier_color(model)
-        print(
-            col + f"{label:<{W[0]}}" + NC
-            + f" {fmt_tokens(inp, outp, cache):<{W[1]}}"
-            + f" {time_str:<{W[2]}}"
-            + f" {fmt_cost(actual):<{W[3]}}"
-            + f" {fmt_cost(vs):<{W[4]}}"
-        )
-        total_inp += inp
-        total_outp += outp
-        total_cache += cache
-        total_actual += actual
-        total_vs += vs
-        max_sec = max(max_sec, sec)
-        total_sec += sec
-
-    saved_pct = round((1 - total_actual / total_vs) * 100) if total_vs else 0
-    total_label = f"TOTAL (saved {saved_pct}%)"
-    total_time = _total_time
-    col = GREEN if total_actual <= 5 else YELLOW
-    print(
-        col + BOLD
-        + f"{total_label:<{W[0]}}"
-        + f" {fmt_tokens(total_inp, total_outp, total_cache):<{W[1]}}"
-        + f" {total_time:<{W[2]}}"
-        + f" {fmt_cost(total_actual):<{W[3]}}"
-        + f" {fmt_cost(total_vs):<{W[4]}}"
-        + NC
-    )
+    rows = [
+        (f"{agent}({model})", model, inp, outp, cache, fmt_time(sec), actual, vs)
+        for agent, model, inp, outp, cache, sec, actual, vs in costs
+    ]
+    render_cost_table(rows, total_time_str, _pricing.baseline_label())
     return 0

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes/services/rendering.py

@@ -139,7 +139,8 @@ def generate_agent_files(agents_config: Dict[str, Any], tiers: Dict[str, Any],
 
         # Expand shared-content include directives (<!-- include: NAME -->)
         # No-op if shared/ directory is absent.
-        prompt_content = expand_includes(prompt_content, AGENTS_DIR / "shared")
+        _agent_include_skip = set() if user_config.get("cost_report_enabled", False) else {"cost_reporting"}
+        prompt_content = expand_includes(prompt_content, AGENTS_DIR / "shared", skip=_agent_include_skip)
 
         # Substitute {{MEMORY_PATH}} with the configured vault/memory path.
         _st = _load_state_fn()

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-notes
-Version: 2.25.0
+Version: 2.27.0
 Summary: AI agent configuration manager for Claude Code, OpenCode, and Copilot
 Author-email: Eugene Naumov <min.verkligheten@gmail.com>
 License-Expression: MIT

{agent_notes-2.25.0 → agent_notes-2.27.0}/agent_notes.egg-info/SOURCES.txt

@@ -125,6 +125,7 @@ agent_notes/data/skills/rails/models.md
 agent_notes/data/skills/rails/testing.md
 agent_notes/data/skills/rails/views.md
 agent_notes/data/skills/refactoring-protocol/SKILL.md
+agent_notes/data/skills/rsi/SKILL.md
 agent_notes/data/skills/setup-project-context/SKILL.md
 agent_notes/data/skills/tdd/SKILL.md
 agent_notes/data/skills/to-issues/SKILL.md
@@ -206,6 +207,7 @@ tests/functional/commands/test_regenerate_command.py
 tests/functional/commands/test_uninstall_command.py
 tests/functional/commands/test_validate_command.py
 tests/functional/memory/__init__.py
+tests/functional/memory/test_memory_add_local_backend.py
 tests/functional/memory/test_memory_command.py
 tests/functional/scripts/__init__.py
 tests/functional/scripts/test_release_script.py
@@ -241,6 +243,7 @@ tests/unit/scripts/__init__.py
 tests/unit/scripts/test_cost_report.py
 tests/unit/scripts/test_cost_report_scoping.py
 tests/unit/scripts/test_formatting_tty.py
+tests/unit/scripts/test_opencode_backend_pricing.py
 tests/unit/scripts/test_time_aggregation.py
 tests/unit/services/__init__.py
 tests/unit/services/test_build_functions.py