Repository navigation
[Issue #159] SupaConsole pilot setup and provider templates, documentation only (v2.1.3 P7) - #160
Conversation
…s (v2.1.3 P7) Documentation only. Adds the Kilo REVIEW.md and CodeRabbit YAML fragments under docs/templates/ (verbatim from provider strategy 3 and 4, with their activation limits) and the SupaConsole drafts under docs/pilot/supaconsole/: PROJECT_CONTEXT.draft.md, dev.json.example and an owner checklist. Every owner action is an unchecked step; nothing is applied to SupaConsole or to any provider. Closes #159 Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
|
|
||
| ## Verification Gate | ||
|
|
||
| The commands of the CI `build` job (`.github/workflows/ci.yml`), in CI order after |
There was a problem hiding this comment.
WARNING: CI command order claim contradicted by gate table and observed CI
Line 27 states the commands are "in CI order after npm ci," but the bullet list below lists lint -> tsc -> prisma generate -> test, while the gate table on line 40 lists prisma generate -> tsc -> lint -> test, matching the observed CI order in README.md line 28. The owner may run commands in the wrong order when setting up SupaConsole.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (5 files)
Previous Review Summary (commit e72de4e)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit e72de4e)Status: 1 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)CRITICAL
WARNING
SUGGESTION
Files Reviewed (5 files)
Reviewed by nemotron-3.5-super-vl:free · Input: 44.9K · Output: 6.9K · Cached: 108.4K Review guidance: REVIEW.md from base branch |
prisma generate, tsc --noEmit, lint, then tests, matching the gate table and the observed ci.yml order. Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
https://entire.com/gh/khaentertainment/-/trails/4/changes/showrunner/1
Closes #159
What this PR does
Documentation only: five new files, nothing under
skills/dev/or the plugin payload.docs/templates/REVIEW.fragment.md: Kilo fragment, text identical to provider strategy §3, plus placement notes and activation limits (§5).docs/templates/coderabbit.fragment.yaml: CodeRabbit fragment, block identical to provider strategy §4 withauto_review.enabled: truestated, plus activation limits (§5).docs/pilot/supaconsole/PROJECT_CONTEXT.draft.md: Verification Gate = CIbuildjob commands; Acceptance Checkops/e2e.sh verify; External Review Policy (Requiredcoderabbit, Ignoredkilo, auto requestscoderabbitonly); T2 pathsprisma/**,src/app/api/auth/**.docs/pilot/supaconsole/dev.json.example: both repositoriesKHAEntertainment/SupaConsole; not named.dev.json.docs/pilot/supaconsole/README.md: owner checklist (all unchecked), facts re-read read-only on 2026-10-27, observed CodeRabbit evidence (labelled not a qualification run).What is NOT done (owner actions, listed as unchecked steps in the README)
Confirming Issues on SupaConsole; any SupaConsole commit or PR (
PROJECT_CONTEXT.md,.dev.json, YAML); any CodeRabbit or Kilo setting; the clean-PR completion qualification; installing the Skill; launching the pilot.Findings the owner should read (README, "Observed CodeRabbit evidence")
mainis6f2b8c5, notb720c68; CIbuildnow usesnpm run lint/npm test(same tools).CHANGES_REQUESTED, andAPPROVEDon clean docs-only PRs docs: record the identity-as-capability and guard-as-lint decisions (ADR-008, ADR-009) #21, Make the Verification Gate a committed script that cannot report success on checks it never ran #22 bound to head).request_changes_workflow: false; committing it without inheritance would replace the organization setting and could remove those review objects. Fragment left verbatim; flagged for the owner (README step 5).Acceptance criteria
docs/templates/, identical to §3/§4, activation limits stated.coderabbitas the one required reviewer; project's own requirements preserved by statement.skills/dev/or the plugin payload changed.Checks
python3 scripts/validate_skill.py: exit 0bash scripts/check_plugin_root.sh: exit 0 (payload unchanged)Test changes and why: none
🤖 Generated with Claude Code