Skip to content

feat(cli-bundle): prototype opt-in Hindsight agent-memory step#14

Open
govtech42 wants to merge 7 commits into
mainfrom
claude/amazing-keller-EkQvk
Open

feat(cli-bundle): prototype opt-in Hindsight agent-memory step#14
govtech42 wants to merge 7 commits into
mainfrom
claude/amazing-keller-EkQvk

Conversation

@govtech42
Copy link
Copy Markdown
Owner

Summary

Prototypes an opt-in Hindsight agent-memory step in cli-bundle, modeled on the OpenViking step (05c-openviking.sh). Off by default; intended for comparative testing against OpenViking (tracked in the TODO Radar from #13).

Hindsight (vectorize-io/hindsight) differs from OpenViking: it's a Dockerized server (API :8888, UI :9999) plus client libraries (npm/pip), with LLM providers including anthropic. The step reflects that shape.

Behavior

05d-hindsight.sh (gated on INSTALL_HINDSIGHT=true):

  1. Installs the client@vectorize-io/hindsight-client (npm, default on); optionally hindsight-client (Python) via pipx, falling back to pip3 --user (PEP 668-safe).
  2. Server (opt-in again) — when HINDSIGHT_START_SERVER=true:
    • no Docker → warns + prints the docker run command;
    • no HINDSIGHT_LLM_API_KEY → warns + skips;
    • otherwise → docker run -d the image with the LLM provider/key and a persistent volume.
    • when false → prints the manual docker run for later.

Safe to run unconfigured: with everything default it just installs the npm client and prints hints — no Docker calls, no secrets required.

Changes

  • profiles/cli-bundle/05d-hindsight.sh (new) — client install + optional Docker server.
  • profiles/cli-bundle/install.sh — wire step after 05c-openviking.sh; header comment.
  • profiles/cli-bundle/.env.exampleINSTALL_HINDSIGHT + HINDSIGHT_NPM_CLIENT / HINDSIGHT_PIP_CLIENT / HINDSIGHT_START_SERVER / HINDSIGHT_LLM_PROVIDER / HINDSIGHT_LLM_API_KEY.
  • README.md / profiles/cli-bundle/README.md — document the prototype step.

Verification

  • bash -n + shellcheck -S warning -e SC1090,SC1091 on the new/edited scripts — OK
  • tests/lint.sh and tests/check_env_completeness.sh — OK

Notes

  • Prototype — not yet integrated into Claude Code as a memory backend; that would be a follow-up once we've evaluated it vs OpenViking.
  • Default LLM provider is openai to match the rest of the bundle, but anthropic is available per upstream.
  • Per request: opening as draft, not merging.

https://claude.ai/code/session_01CMLyxQST7SD4tSDvE1nF9T


Generated by Claude Code

claude added 7 commits May 29, 2026 13:44
Adds OpenViking (@openviking/cli, `ov`) as a sixth coexisting CLI in the
cli-bundle profile, following the npm-install pattern used by Codex.

- New 05c-openviking.sh: npm global install, INSTALL_OPENVIKING toggle,
  prints `ov config` setup hint (server defaults to http://localhost:1933)
- Wire into cli-bundle/install.sh after 05b-opencode
- Add INSTALL_OPENVIKING=false toggle + auth note to .env.example
- Document in README (profile table, CLI toggle table, repo layout)

Upstream: https://github.com/volcengine/OpenViking
Drops the GSD (Get Shit Done) and gstack plugins from the cli-bundle
plugin step. superpowers and OpenSpec remain, along with the official
Anthropic marketplace plugins.

- 09-plugins.sh: remove GSD and gstack install blocks; drop now-unused
  base-packages.sh source and ~/.bun/bin from PATH
- lib/plugins.sh: remove install_gstack_for(); tidy install_claude_plugin
  doc comment
- lib/base-packages.sh: remove install_bun() (only gstack consumed it)
- .env.example: drop INSTALL_GSD, INSTALL_GSTACK, GSTACK_TARGETS toggles
- README.md / cli-bundle/README.md: update plugin tables and lib helper lists
- .claude/TODO.md: drop GSD/gstack follow-ups
Wires the @tech-leads-club/agent-skills CLI into the plugins step as an
opt-in, non-interactive install across the CLIs that support it
(claude-code, codex, cursor, opencode). Skills install globally so every
project sees them.

Ships a curated default skill set tuned for the MiranteGov / Municipium
stack (Next.js + React Native, NestJS + Nx monorepo, GovTech accessibility
+ security): 29 skills spanning a11y, security review/threat-model, NestJS
modular monolith + tactical DDD, Nx tooling, React/frontend quality, perf,
Sentry, and ADR/RFC/TDD docs.

- lib/plugins.sh: install_agent_skills() helper (npx, --global, repeated
  --skill/--agent flags)
- 09-plugins.sh: INSTALL_AGENT_SKILLS block; resolves target agents to the
  intersection of installed CLIs and AGENT_SKILLS_AGENTS; baked-in curated
  default for AGENT_SKILLS_LIST
- .env.example: INSTALL_AGENT_SKILLS, AGENT_SKILLS_AGENTS, AGENT_SKILLS_LIST
- README.md / cli-bundle/README.md: document the registry and curated set

All 29 curated skill names validated against the upstream catalog.
Tracks vectorize-io/hindsight for later evaluation as an alternative/
complement memory backend (same category as OpenViking) for the
cli-bundle agents. Notes install paths (Docker/pip/npm + embedded) and
upstream link.
Adds 05d-hindsight.sh modeled on the OpenViking step: opt-in via
INSTALL_HINDSIGHT, sources .env, gates cleanly, and prints next-step
hints. Installs the Hindsight client (npm by default, pip optional via
pipx/pip --user) and can start the Dockerized server (API :8888, UI
:9999) when HINDSIGHT_START_SERVER=true and Docker + an LLM key are
present; otherwise prints the docker run command for manual start.

- 05d-hindsight.sh: client install + optional Docker server
- install.sh: wire step after 05c-openviking; header comment
- .env.example: INSTALL_HINDSIGHT + HINDSIGHT_* config block
- README.md / cli-bundle/README.md: document the prototype step

Prototype only — not yet integrated into Claude Code as a memory
backend; tracked for comparative testing against OpenViking.
@govtech42 govtech42 requested a review from Copilot May 29, 2026 21:53
@govtech42 govtech42 marked this pull request as ready for review May 29, 2026 21:53
@govtech42 govtech42 requested review from Copilot and removed request for Copilot May 29, 2026 21:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an opt-in 05d-hindsight.sh step to the cli-bundle profile that installs the Hindsight agent-memory client (npm by default, optional Python via pipx/pip) and optionally starts a Dockerized server. Wires the new step into install.sh after OpenViking and documents the env vars and behavior in both READMEs and .env.example. The step is gated by INSTALL_HINDSIGHT (off by default) and the server start is gated again by HINDSIGHT_START_SERVER plus presence of Docker and an LLM API key, mirroring the OpenViking precedent.

Changes:

  • New profiles/cli-bundle/05d-hindsight.sh with client install (npm/pipx/pip) and optional docker run server start, including manual-run hints on missing Docker/API key.
  • New Hindsight env toggles in profiles/cli-bundle/.env.example and wiring in install.sh.
  • README updates (root and profiles/cli-bundle/README.md) describing the prototype step, ports, and toggles.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
README.md Updates the cli-bundle step chain diagram and adds a Hindsight bullet to the cli-bundle stack description.
profiles/cli-bundle/README.md Adds a Portuguese-language section documenting the Hindsight prototype step, env vars, and Docker run command.
profiles/cli-bundle/install.sh Updates header comment to mention Hindsight and invokes the new 05d-hindsight.sh after OpenViking.
profiles/cli-bundle/05d-hindsight.sh New opt-in step: installs npm client (default), optional Python client via pipx/pip3, and optionally starts the Hindsight Docker server with safety gates.
profiles/cli-bundle/.env.example Adds INSTALL_HINDSIGHT, HINDSIGHT_NPM_CLIENT, HINDSIGHT_PIP_CLIENT, HINDSIGHT_START_SERVER, HINDSIGHT_LLM_PROVIDER, HINDSIGHT_LLM_API_KEY with comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants