Skip to content

docs: Add a memory bank for agent architecture notes - #267

Merged
schloerke merged 1 commit into
schloerke/memory-bank-agents.md-initfrom
schloerke/memory-bank
Nov 2, 2026
Merged

schloerke merged 1 commit into
schloerke/memory-bank-agents.md-initfrom
schloerke/memory-bank

Conversation

@schloerke

@schloerke schloerke commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

Adds a memory bank, modeled on the one in posit-dev/assistant:

  • memory-bank/ holds architecture notes, each with title and description frontmatter.
  • .agents/skills/memory-bank/ is a skill that tells agents how to find, read, and update those notes. Its list_memory_bank.mjs script prints a catalog of the documents and the heading outline of a single document. Unlike the original, it reads the one-line frontmatter itself instead of using a YAML package, so it runs before npm ci.
  • .claude/skills links to .agents/skills, so Claude Code finds the skill too.
  • AGENTS.md gains a section telling agents to check the memory bank before exploring the code.

Two seed documents:

  • browserTests.md: traps in the pytest/Playwright tests. These are the shared port 8100, stale service workers, which errors each engine makes visible, the race when switching apps, and the silent fallback for unknown URL hashes.
  • pythonPackageVersions.md: examples run against shinylive_lock.json and the packages/ submodules, not py-shiny main.

@schloerke
schloerke added this pull request to stack #268 November 2, 2026 20:03
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Unit tests only — the pure-logic modules in src/. The React components, hooks, engine proxies and service worker are covered by the pytest + Playwright suites in tests/, which are not measured here.

Coverage report

St.❔
Category Percentage Covered / Total
🟢 Statements 100% 1994/1994
🟢 Branches 100% 400/400
🟢 Functions 100% 96/96
🟢 Lines 100% 1994/1994

Test suite run success

342 tests passing in 20 suites.

Report generated by 🧪jest coverage report action from 7fc76f2

@schloerke
schloerke removed this pull request from stack #268 November 2, 2026 21:36
@schloerke
schloerke added this pull request to stack #272 November 2, 2026 21:37
@schloerke
schloerke merged commit 882e926 into main Nov 2, 2026
16 checks passed
@schloerke
schloerke deleted the schloerke/memory-bank branch November 2, 2026 21:45
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.

1 participant