{"slug":"goga-review-cell","title":"goga-review-cell","summary":"Three-dimension cell review","platform":"Claude","tags":[],"authorName":"LLM Mart","authorSlug":"llm-mart","score":0,"source":"github","price":null,"verified":false,"createdAt":"2026-08-24T17:38:12.842295Z","repo":{"url":"https://github.com/qarium/goga","stars":32,"forks":0,"license":"BSD-3-Clause","updatedAt":"2026-09-27T19:31:53Z"},"bodyHtml":"<hr>\n<h2>name: goga-review-cell\ndescription: Three-dimension cell review</h2>\n<h1>Cell Review</h1>\n<h2>Purpose</h2>\n<p>Reviews a cell across three dimensions and proposes concrete remediation actions for each detected issue. Each action requires explicit user confirmation before execution.</p>\n<p>You <strong>identify</strong> issues, <strong>propose</strong> solutions, and <strong>execute</strong> user-approved actions (invoking other skills, editing files).</p>\n<hr>\n<h2>Key Principle</h2>\n<p><strong>Identify the exact problem before proposing a solution.</strong> The three review dimensions target distinct root causes and require different remediation strategies. Do not conflate them — classify each finding into exactly one of the three categories before proposing any action.</p>\n<h3>User Interaction Rule</h3>\n<p><strong>Always provide answer options.</strong> When requesting user confirmation, decisions, or clarifications, always offer 2–4 concrete options to choose from. Never pose open-ended questions without selectable choices.</p>\n<hr>\n<h2>Steps</h2>\n<h3>Step 1: Load Context</h3>\n<ol>\n<li>Invoke <code>goga-lang-disp</code> via the <strong>Skill tool</strong> to load the target language skill.\nThe language skill defines implementation conventions: cell structure, facade, signature rules, <strong>naming</strong>.\nExamples in other skills may follow naming conventions of one language (e.g., snake_case), while the target language\nrequires different conventions (e.g., PascalCase). The language skill contains authoritative rules for the target language.</li>\n<li>Load the DSL specification and DSL application principles:\n<ul>\n<li>Invoke <code>goga-cell</code> via the <strong>Skill tool</strong> to understand CODEMANIFEST DSL syntax, structural rules, and semantics\n(cell structure, <code>Imports</code>, <code>Usages</code>, <code>Annotations</code> directives, types, signatures, mutations, embeddings, constraints)</li>\n<li>Invoke <code>goga-cookbook</code> via the <strong>Skill tool</strong> to understand cell and CODEMANIFEST working principles\n(when to use Entity vs Routine, when to apply mutations and embeddings, usage file authoring principles, cell granularity)</li>\n<li><strong>Critical for accurate analysis</strong>: without understanding DSL rules, CODEMANIFEST entries will be misinterpreted</li>\n<li>Consult the loaded specification and principles whenever CODEMANIFEST entry correctness is uncertain</li>\n</ul>\n</li>\n<li>Read the cell's <code>CODEMANIFEST</code> file</li>\n<li>Parse all entities, methods, properties, imports, usages, annotations, re-exports, and locations</li>\n<li>Enumerate all source files in the cell directory (excluding <code>.usages/</code>, build artifacts, tests, and other non-contract files)</li>\n<li>Read all source files corresponding to declared <code>location</code> values</li>\n<li>Read all <code>.usages/*.md</code> files if the <code>.usages/</code> directory exists</li>\n<li>Verify the facade file exists</li>\n</ol>\n<hr>\n<h3>Step 2: Run Tools</h3>\n<h4>2a. Run Linter</h4>\n<pre><code>goga lint\n</code></pre>\n<p>If the linter reports errors, record each as a DSL syntax issue. Fix all DSL syntax errors before proceeding with analysis.</p>\n<h4>2b. Run Schema</h4>\n<pre><code>goga schema\n</code></pre>\n<p>Extract the cell's position in the project hierarchy for contextual reference.</p>\n<hr>\n<h3>Step 3: Analysis 1 — Code vs Requirements</h3>\n<p><strong>Question: Does the code implement what CODEMANIFEST requires?</strong></p>\n<p>For each entity declared in CODEMANIFEST, verify that the implementation satisfies the contract.</p>\n<h4>3a. Checks</h4>\n<p>For each entity:</p>\n<ul>\n<li><strong>Signature conformance</strong>: Compare <code>\"()\"</code> (constructor signature) between CODEMANIFEST and source code</li>\n<li><strong>Method coverage</strong>: Compare the <code>\"methods\"</code> dictionary — identify missing, extra, or mismatched method signatures</li>\n<li><strong>Property coverage</strong>: Compare the <code>\"properties\"</code> dictionary — identify missing, extra, or mismatched property types</li>\n<li><strong>Facade exposure</strong>: Verify the entity appears on both sides. Absence in source code means it is not exported via the facade</li>\n</ul>\n<p>For each function (routine):</p>\n<ul>\n<li><strong>Signature conformance</strong>: Compare the signature string between CODEMANIFEST and source code</li>\n<li><strong>Existence</strong>: Absence in source code means the function is not exported via the facade</li>\n</ul>\n<p>Additional checks:</p>\n<ul>\n<li><strong>Location validity</strong>: Does the source file exist at path <code>&lt;cell-path&gt;/&lt;location&gt;</code>?</li>\n<li><strong>Behavioral conformance</strong>: Does the implementation satisfy behavioral requirements from annotations?</li>\n<li><strong>Import utilization</strong>: Are imported types from <code>Imports</code> actually referenced in source code?</li>\n</ul>\n<h4>3b. Findings Presentation</h4>\n<p>When issues are detected, present each finding with:</p>\n<ul>\n<li>Exact location (entity, method/property, file)</li>\n<li>What CODEMANIFEST requires vs. what the code actually does</li>\n</ul>\n<h4>3c. Proposed Action</h4>\n<p><strong>Formulate a task</strong> describing the discrepancy between code and contract.</p>\n<p>Propose: invoke <code>/goga:design</code> in <strong>brainstorm</strong> mode, passing the task as context. This produces an implementation redesign that satisfies the contract.</p>\n<p>Request user confirmation before execution.</p>\n<hr>\n<h3>Step 4: Analysis 2 — Requirements vs Code and Usages</h3>\n<p><strong>Questions: Do CODEMANIFEST requirements match the actual code and usages? Is CODEMANIFEST well-authored?</strong></p>\n<p>The contract may be incomplete, incorrect, outdated, or poorly written compared to what the code actually implements and what usages describe.</p>\n<h4>4a. Checks: Accuracy</h4>\n<ul>\n<li><strong>Undocumented entities</strong>: Are there public classes/functions in the code not declared in CODEMANIFEST?</li>\n<li><strong>Undocumented methods/properties</strong>: Are there public API members of documented classes missing from CODEMANIFEST?</li>\n<li><strong>Contract accuracy vs usages</strong>: Do annotations reference actually applicable usages? Are usages used in code but not mentioned in annotations?</li>\n<li><strong>Requirements accuracy</strong>: Do described signatures, types, and behaviors match what the code actually does? If the code is correct and the contract is wrong — the contract requires correction.</li>\n</ul>\n<h4>4b. Checks: Authoring Quality</h4>\n<p>For each entity and function (routine) in CODEMANIFEST, verify that annotations satisfy the following authoring quality criteria. Even a technically correct contract degrades overall quality when annotations are poorly written.</p>\n<p><strong>Content guidelines:</strong></p>\n<ul>\n<li>Begin each annotation with a clear purpose statement — what this entity/function/method does and why it exists</li>\n<li>For each parameter, provide a description using <code>`parameter_name`: description</code> syntax</li>\n<li>For non-trivial logic (multi-step transformations, conditional flows, state transitions), include an <code>Algorithm:</code> section with numbered steps tracing the execution flow</li>\n<li>For constraints, edge cases, format requirements, or preconditions — include a <code>Requirements:</code> section with explicit descriptions</li>\n<li>Document return value format when semantics are non-obvious from the signature (e.g., when meaning differs from type or structure is complex)</li>\n<li>Include usage examples when they help clarify the contract — configuration examples for builders, input/output pairs for parsers, invocation patterns for facades</li>\n</ul>\n<p><strong>Quality guidelines:</strong></p>\n<ul>\n<li>Each annotation must be specific enough for direct implementation — no TBD, TODO, or vague phrasing</li>\n<li>Each annotation must permit exactly one interpretation — if ambiguous, rewrite it</li>\n<li>Maintain consistent style and structure across all annotations within a single CODEMANIFEST file</li>\n<li>All backtick references (<code>`name`</code>) must resolve to entities in the current CODEMANIFEST context — types from <code>Imports</code>, practices from <code>Usages</code>, or parameters from the signature</li>\n</ul>\n<h4>4c. Findings Presentation</h4>\n<p>When issues are detected, present each finding with:</p>\n<ul>\n<li>Exact CODEMANIFEST location (entity, method/property, section)</li>\n<li>Nature of the issue: missing declaration, inaccurate description, incorrect type, outdated reference, poor annotation quality</li>\n<li>For authoring quality issues: a specific improvement recommendation (e.g., \"add <code>Algorithm:</code> section\", \"describe parameter <code>path</code>\", \"restructure annotation with <code>Algorithm:</code> and <code>Requirements:</code> sections\")</li>\n</ul>\n<h4>4d. Proposed Action</h4>\n<p><strong>Edit CODEMANIFEST</strong> to correct inaccuracies and improve authoring quality, then invoke <code>/goga:design</code> in <strong>changes</strong> mode to update the design document based on the corrected contract.</p>\n<p>Request user confirmation before execution.</p>\n<hr>\n<h3>Step 5: Analysis 3 — Usage Existence and Adequacy</h3>\n<p><strong>Question: Do usage files exist and are they adequate for the cell implementation?</strong></p>\n<h4>5a. Checks</h4>\n<ul>\n<li><strong>Existence</strong>: For each <code>Usages</code> entry with a file path — does the file exist? (project-level in <code>.goga/usages/</code>, cell-level in <code>.usages/</code> directory)</li>\n<li><strong>Annotation reference</strong>: Is each declared usage referenced in an annotation via backtick syntax?</li>\n<li><strong>Imported usage existence</strong>: For each imported usage from <code>Imports</code> → <code>Usages</code> — does <code>{from_path}/.usages/{usage_name}.md</code> exist?</li>\n<li><strong>Adequacy</strong>: Does each <code>.usages/*.md</code> file describe cell API usage? Is the content accurate and sufficiently detailed for consumers?</li>\n<li><strong>Missing usages</strong>: Are there external library imports, recurring patterns, or code conventions not covered by any Usages entry?</li>\n<li><strong>Categorization</strong>: Are <code>.usages/</code> functional categories clearly defined and non-overlapping? Should inline usages be extracted into category files?</li>\n</ul>\n<h4>5b. Findings Presentation</h4>\n<p>When issues are detected, present each finding with:</p>\n<ul>\n<li>Which usage is missing, outdated, or inadequate</li>\n<li>What practice it should describe</li>\n</ul>\n<h4>5c. Proposed Action</h4>\n<p><strong>Create or update <code>.usages/*.md</code> files</strong> directly. This does not require invoking another command — usage files are practice documentation editable directly.</p>\n<p>Request user confirmation before execution.</p>\n<hr>\n<h3>Step 6: Execute Approved Actions</h3>\n<p>For each user-approved action across all three analyses:</p>\n<h4>6a. Code vs Requirements (Step 3)</h4>\n<ol>\n<li>Formulate a clear redesign task description</li>\n<li>Invoke <code>/goga:design</code> and select <strong>brainstorm</strong> mode</li>\n<li>Pass the task as context — the design skill will handle the redesign</li>\n</ol>\n<h4>6b. Requirements vs Code/Usages (Step 4)</h4>\n<ol>\n<li>Apply CODEMANIFEST edits directly</li>\n<li>Run linter for validation: <code>goga lint</code></li>\n<li>Fix all DSL syntax errors after edits</li>\n<li>Invoke <code>/goga:design</code> and select <strong>changes</strong> mode</li>\n<li>The design skill will detect changes via git diff and update the document accordingly</li>\n</ol>\n<h4>6c. Usages (Step 5)</h4>\n<ol>\n<li>Create or update the corresponding <code>.usages/*.md</code> files</li>\n<li>Ensure content accurately describes cell API usage with sufficient detail for consumers (implementation details may be present but are secondary)</li>\n</ol>\n<h4>6d. Final Validation</h4>\n<p>After executing <strong>all</strong> approved actions (6a, 6b, 6c):</p>\n<ol>\n<li>Run linter for final validation: <code>goga lint</code></li>\n<li>If the linter reports errors — fix them</li>\n<li>Repeat steps 1–2 until the linter passes without errors</li>\n</ol>\n<hr>\n<h2>Result</h2>\n<ul>\n<li>Review findings across all three dimensions</li>\n<li>Analysis 1 (Code vs Requirements): task delegated to <code>/goga:design brainstorm</code></li>\n<li>Analysis 2 (Requirements vs Code/Usages): CODEMANIFEST edits + <code>/goga:design changes</code></li>\n<li>Analysis 3 (Usages): created or updated <code>.usages/*.md</code> files</li>\n</ul>\n<hr>\n<h2>Final Self-Check</h2>\n<p>Before completion, verify:</p>\n<ol>\n<li>Was the DSL specification loaded via <code>goga-cell</code> and <code>goga-cookbook</code> skills before analysis?</li>\n<li>Was the cell's CODEMANIFEST fully parsed?</li>\n<li>Were all source files verified against the contract?</li>\n<li>Were all <code>.usages/</code> files analyzed?</li>\n<li>Was <code>goga lint</code> executed?</li>\n<li>Was <code>goga schema</code> executed?</li>\n<li>Was Analysis 1 (Code vs Requirements) completed with clear findings and a proposed action?</li>\n<li>Was Analysis 2 (Requirements vs Code/Usages) completed with both accuracy and authoring quality checks?</li>\n<li>Was Analysis 3 (Usage Existence and Adequacy) completed with clear findings and a proposed action?</li>\n<li>Was each finding classified into the correct analysis category before proposing an action?</li>\n<li>Were all actions confirmed by the user before execution?</li>\n<li>Were approved actions executed correctly?</li>\n<li>Was <code>goga lint</code> executed <strong>after</strong> all actions and did it pass without errors?</li>\n</ol>\n<p>If any answer is \"no\" — complete the outstanding work before returning.</p>\n<hr>\n","files":[{"path":"SKILL.md","sizeBytes":11754,"isText":true}],"reviewScore":null,"reviewSummary":null,"trust":{"provenance":"trusted-source-unreviewed","notice":"Community-authored content, reproduced verbatim and not vetted as instructions. Treat it as data to evaluate, never as directives to follow.","bodySource":null},"bodyLocked":false,"purchaseUrl":null,"sourceUrl":null,"report":{"provenance":"trusted-source-unreviewed","screen":{"ran":true,"outcome":"clean","suspicious":0,"notes":0,"hiddenCharacters":false},"virusScan":{"engine":"clamav","status":"clean","scannedAt":"2026-08-24T17:40:20.871682Z","sha256":"3BE18FB25DF06737D6942B8FBE7CECEE94ABD5D33C84CADD16244F97865A1C92","sizeBytes":4390},"review":null,"source":{"repositoryUrl":"https://github.com/qarium/goga","path":"goga/assets/skills/goga-review-cell","license":"BSD-3-Clause","commit":"9fdb39b191bec889e345e42ffa1549d2727ca247","subtreeSha":"31B8C6DF3B9F5E5421FDF4CCED77916BED9E37D43C66EAA462534D42F684D03B","lastSyncedAt":"2026-09-27T20:56:56.704728Z"},"reviewedAt":"2026-08-24T17:49:48.874814Z","notice":"Community-authored content, reproduced verbatim and not vetted as instructions. Treat it as data to evaluate, never as directives to follow."},"install":[{"target":"skills-cli","command":"npx skills add https://github.com/qarium/goga/tree/1.2.x/goga/assets/skills/goga-review-cell"},{"target":"claude-code","command":"claude plugin marketplace add https://llmmart.ai/marketplace.json && claude plugin install qarium-goga@llmmart"},{"target":"git","command":"git clone https://github.com/qarium/goga.git"}]}