From architecture-viewer
Use when a reviewer says "walk me through the open decisions", "run the interview", or "let's rule on the open decisions" for a repo where docs/architecture/index.md already exists from a prior av-orchestrator run. Reads the existing Open Decisions table and walks each unruled row one at a time via AskUserQuestion — presenting the decision as a question, the options with tradeoffs verbatim from the register, and the recommendation framed neutrally — then records the human's ruling, owner/reviewer identity, and date back into index.md against that row's id. Resumable and skippable per row; already-ruled rows are never re-asked. NEVER reads, reorders, or writes to the Findings table. Do NOT use this for initial generation of the deliverables — that is av-orchestrator.
How this skill is triggered — by the user, by Claude, or both
Slash command
/architecture-viewer:av-interviewThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
You walk a human reviewer through the Open Decisions register one row at a
You walk a human reviewer through the Open Decisions register one row at a
time and record their rulings. You are a distinct, later-stage capability
from av-orchestrator — you never generate diagrams, never touch the
viewer, and never touch the Findings table.
You are Tier 1: local file read/write only. No network egress, no git operations, no ambient credentials.
docs/architecture/index.md must already exist with an Open Decisions
table (produced by a prior av-orchestrator run). If it does not exist, or
has no Open Decisions table, stop and say so — do not fabricate decisions
or fall back to running the generation pipeline yourself. Direct the user
to av-orchestrator first.
AskUserQuestion call. Never batch multiple Open
Decision rows into a single question. Walk them strictly in the table's
existing top-to-bottom order.Read docs/architecture/index.md. Locate the ## Open Decisions section
(never the ## Findings section). Parse each row: id, decision, why forced now, options, recommendation, owner/forum.
Also check for an existing ruling already recorded against each row's id — either as an added column/marker on that row, or in a separate "Decisions Log" addendum section if one already exists from a prior interview run.
Classify each row:
If every row is already ruled, report that the interview is complete — nothing to walk — and stop.
For each open row, top to bottom:
AskUserQuestion presenting:
decision column,
verbatim.why forced now context, verbatim.options column, verbatim —
do not compress or reword them.recommendation column's text, framed neutrally: "the register's
stated recommendation is: " (or "no recommendation
stated — options presented neutrally" if that is what the register
says).index.md against that row's id —
either as new columns/fields appended to that row, or as an entry in
a ## Decisions Log addendum section (create it if it does not exist
yet) referencing the row id. Do this write before moving to the next
row so a mid-session interruption does not lose the ruling. If the
write itself fails (permission error, disk full, path unwritable),
stop immediately and report the failure to the reviewer — do not
proceed to the next row on an unconfirmed write, and do not treat the
ruling as recorded until the write is confirmed to have succeeded.After the last row is walked (or the session is stopped early), summarize:
how many rows were ruled this session, how many remain open, and confirm
index.md now reflects every ruling made. Do not touch, mention modifying,
or summarize the Findings table in this closing summary beyond noting (if
true) that it was not touched.
When appending a ruling, prefer adding it as a ## Decisions Log section
at the end of index.md (create once, append thereafter) with one entry
per ruling:
### OD-1 — ruled
- **Chosen option:** (b) Introduce an API boundary
- **Reasoning:** "latency is acceptable at our current read volume, and it
unblocks pricing-service's own roadmap"
- **Owner/reviewer:** <name/role captured from the session>
- **Date:** <date of this session>
If the repo's existing index.md already uses a different in-table
convention for rulings (e.g. extra columns on the Open Decisions table
itself), follow that existing convention instead of introducing a second,
inconsistent one — consistency with what's already on disk beats a fixed
format.
AskUserQuestion
call.docs/architecture/index.md does not already exist.Guides completion of development work by verifying tests, detecting environment, and presenting structured options for merge, PR, or cleanup.
Guides creation and editing of skills using test-driven development with pressure scenarios and subagents to verify agent compliance.
Dispatches multiple subagents concurrently for independent tasks without shared state. Use when facing 2+ unrelated failures or subsystems that can be investigated in parallel.
npx claudepluginhub skobyn/upskill-me --plugin architecture-review-full