Skip to content

feat(insights): read-only insights + journaling reporting surface #149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
eaitbrahim merged 2 commits into from
Jul 24, 2026
Merged

feat(insights): read-only insights + journaling reporting surface #149

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d7a7170
feat(insights): add keel insights summary + journal read-only reporting
eaitbrahim Jul 24, 2026
95d7d7b
fix(insights): correct include-open count line and quantize human render
eaitbrahim Jul 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 10 additions & 12 deletions keel/cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@
`subscription attest|set|show` (the per-venue, user-attested allowance rail 14 reads live),
`status` (`commands.status`: the read-only, no-broker operator dashboard the paper-mode-fidelity
spec deferred -- mode/kill-switch/autonomy/Rail 11/positions/rules/data freshness, plus `--json`),
and a Phase-4 `insights` stub.
and `insights summary|journal` (`commands.insights`: a read-only VIEW over the same substrate --
per-rule promotion-gate distance and a filterable trade journal, also with `--json`).

**Dangerous commands ask a human; nothing needs a stored secret.** The former scrypt passphrase
gate is gone (see `2026-07-21-security-simplification-design.md`). Five commands re-permit trading
Expand Down Expand Up @@ -79,6 +80,7 @@
from keel.commands._products import _default_sim_products, _history_product
from keel.commands.autonomy import autonomy_group
from keel.commands.db import db_group
from keel.commands.insights import insights_group
from keel.commands.rules import rules_group, rules_seed
from keel.commands.status import status_cmd
from keel.commands.subscription import subscription_group
Expand Down Expand Up @@ -1647,6 +1649,13 @@ def simulate(
cli.add_command(tui_cmd)


# -- insights (read-only promotion-gate + journal reporting, no broker call) --------------------

# A pure VIEW over `gather_status`/`StatusReport`, the repository read methods, and the
# promotion/track-record machinery -- defined in `keel.commands.insights` and registered here.
cli.add_command(insights_group)


# -- kill / resume ------------------------------------------------------------------------------


Expand Down Expand Up @@ -1780,16 +1789,5 @@ def reset_hwm(ctx: click.Context) -> None:
click.echo("equity high-water mark reset: it will re-seed from the next cycle's equity.")


# -- insights (Phase 4 stub) ---------------------------------------------------------------------


@cli.command()
@click.pass_context
@with_disclaimer
def insights(ctx: click.Context) -> None:
"""Portfolio insights (Phase 4 -- not yet implemented)."""
click.echo("insights: not yet implemented (see Phase 4 of the roadmap).")


if __name__ == "__main__": # pragma: no cover
cli()
Loading
Loading