Skip to content

docs: ARCHITECTURE.md shows PiranhaDB as offline - #81

Open
chaksaray wants to merge 1 commit into
developfrom
docs/80-piranha-offline-architecture
Open

chaksaray wants to merge 1 commit into
developfrom
docs/80-piranha-offline-architecture

Conversation

@chaksaray

Copy link
Copy Markdown
Member

Closes #80

What and why

ARCHITECTURE.md labelled PiranhaDB LIVE and named api.piranha.bawbel.io in two places (the Layer 3 box and the mermaid node), while the "piranhadb (Layer 3, offline)" section of the same file says the host does not resolve. The diagram now agrees with that section. Applied the ecosystem classifier (ECOSYSTEM-RUNBOOK STEP 3, D3 approved).

  • Layer 3 box: [LIVE] becomes [OFFLINE], the host line becomes "hosted API offline (host does not resolve)", and the third line points at api.aveproject.org for the corpus. Box width unchanged: 76 characters on all five lines
  • Mermaid node: PiranhaDB<br/>OFFLINE · host does not resolve

Governing spec

DESIGN.md section(s): none. ARCHITECTURE.md only; no behavior change.

Not in this PR: DESIGN.md:838 names the retired AVE host. PR #76 rewrites that section, and DESIGN.md changes merge alone. The data-flow arrows that route through the PiranhaDB node are not rewritten here.

Output contract

  • Changes: ARCHITECTURE.md text, 2 hunks
  • Does not change: code, DESIGN.md, schemas, CLI flags, manifest fields, reason strings, audit record fields
  • New or changed CLI flags, manifest fields, reason strings, audit record fields: none

Verification

pytest                          -> 346 passed in 11.12s
pytest tests/property/ -x       -> 26 passed in 1.36s
pytest tests/replay/            -> 32 passed in 0.36s
pytest tests/test_m4_corpus.py  -> 10 passed in 0.18s   (approval budget)
pre-commit run --all-files      -> ruff, bandit, trim trailing whitespace, fix end of files, check json, check yaml, check for merge conflicts: 7 Passed
git grep -c 'api.piranha.bawbel.io' -- ARCHITECTURE.md  -> 1

The one remaining match is the sentence in the offline section that says the host does not resolve.

Trial merges (git merge, no conflicts): with #78 (docs/77-bawbel-mcp-facts) and with #76 (docs/74-design-ave-1-1-0).

Found while editing, not changed here: the same diagram says "schema 1.2 · 51 records · LIVE" for AVE (ARCHITECTURE.md:63), but AVE publishes schema 1.1.0 and 87 records. That is GATE-4 territory (#71, with #75).

Checklist

  • Every commit carries a DCO sign-off (git commit -s)
  • DESIGN.md is still accurate after this change
  • Replay test in tests/replay/ if this blocks an attack class (not applicable)
  • Docs updated for user-facing changes
  • No em dashes in docs, comments, or error strings

🤖 Generated with Claude Code

The Layer 3 box and the mermaid node labelled PiranhaDB LIVE and named
api.piranha.bawbel.io, while the piranhadb section of the same file says
the host does not resolve. Both now say offline, and the box points at
api.aveproject.org for the corpus, matching that section. The box width
is unchanged. The data-flow arrows are not rewritten here.

Closes #80

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Signed-off-by: chaksaray <chaksaray@gmail.com>

This branch has not been deployed

No deployments
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