@bastani/atomic 0.6.5 → 0.6.6-0

package/src/sdk/workflows/builtin/open-claude-design/opencode/index.ts

@@ -151,7 +151,10 @@ export default defineWorkflow({
             description: "Locate design files and tokens",
           },
           {},
-          { title: "ds-locator" },
+          {
+            title: "ds-locator",
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -171,7 +174,10 @@ export default defineWorkflow({
             description: "Analyze design tokens and patterns",
           },
           {},
-          { title: "ds-analyzer" },
+          {
+            title: "ds-analyzer",
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -191,7 +197,10 @@ export default defineWorkflow({
             description: "Find existing design patterns",
           },
           {},
-          { title: "ds-patterns" },
+          {
+            title: "ds-patterns",
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -214,7 +223,10 @@ export default defineWorkflow({
           description: "Build design system with user approval (HIL)",
         },
         {},
-        { title: "design-system-builder" },
+        {
+          title: "design-system-builder",
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
         async (s) => {
           const result = await s.client.session.prompt({
             sessionID: s.session.id,
@@ -253,7 +265,10 @@ export default defineWorkflow({
               description: "Capture web reference via playwright",
             },
             {},
-            { title: "web-capture" },
+            {
+              title: "web-capture",
+              permission: [{ permission: "*", pattern: "*", action: "allow" }],
+            },
             async (s) => {
               const result = await s.client.session.prompt({
                 sessionID: s.session.id,
@@ -283,7 +298,10 @@ export default defineWorkflow({
               description: "Parse reference document",
             },
             {},
-            { title: "file-parser" },
+            {
+              title: "file-parser",
+              permission: [{ permission: "*", pattern: "*", action: "allow" }],
+            },
             async (s) => {
               const result = await s.client.session.prompt({
                 sessionID: s.session.id,
@@ -317,7 +335,10 @@ export default defineWorkflow({
     await ctx.stage(
       { name: "generator", description: "Generate first design version" },
       {},
-      { title: "generator" },
+      {
+        title: "generator",
+        permission: [{ permission: "*", pattern: "*", action: "allow" }],
+      },
       async (s) => {
         const result = await s.client.session.prompt({
           sessionID: s.session.id,
@@ -351,7 +372,10 @@ export default defineWorkflow({
           description: `Collect refinement feedback (iteration ${iteration})`,
         },
         {},
-        { title: `user-feedback-${iteration}` },
+        {
+          title: `user-feedback-${iteration}`,
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
         async (s) => {
           const result = await s.client.session.prompt({
             sessionID: s.session.id,
@@ -384,7 +408,10 @@ export default defineWorkflow({
             description: `Design critique (iteration ${iteration})`,
           },
           {},
-          { title: `critique-${iteration}` },
+          {
+            title: `critique-${iteration}`,
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -411,7 +438,10 @@ export default defineWorkflow({
             description: `Visual validation (iteration ${iteration})`,
           },
           {},
-          { title: `screenshot-${iteration}` },
+          {
+            title: `screenshot-${iteration}`,
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -450,7 +480,10 @@ export default defineWorkflow({
           description: `Apply refinements (iteration ${iteration})`,
         },
         {},
-        { title: `apply-changes-${iteration}` },
+        {
+          title: `apply-changes-${iteration}`,
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
         async (s) => {
           const result = await s.client.session.prompt({
             sessionID: s.session.id,
@@ -495,7 +528,10 @@ export default defineWorkflow({
           description: "Remove banned anti-patterns before export",
         },
         {},
-        { title: "forced-fix" },
+        {
+          title: "forced-fix",
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
         async (s) => {
           const result = await s.client.session.prompt({
             sessionID: s.session.id,
@@ -539,7 +575,10 @@ export default defineWorkflow({
         description: "Export design and create handoff bundle",
       },
       {},
-      { title: "exporter" },
+      {
+        title: "exporter",
+        permission: [{ permission: "*", pattern: "*", action: "allow" }],
+      },
       async (s) => {
         const result = await s.client.session.prompt({
           sessionID: s.session.id,

package/src/sdk/workflows/builtin/ralph/claude/index.ts

@@ -28,6 +28,7 @@ import { defineWorkflow, extractAssistantText } from "../../../index.ts";
 import {
   buildPlannerPrompt,
   buildOrchestratorPrompt,
+  buildCodeSimplifierPrompt,
   buildInfraDiscoveryPrompts,
   buildReviewPrompt,
   filterActionable,
@@ -94,7 +95,7 @@ export default defineWorkflow({
 
     for (let iteration = 1; iteration <= maxLoops; iteration++) {
       // ── Plan ────────────────────────────────────────────────────────────
-      await ctx.stage(
+      const planner = await ctx.stage(
         { name: `planner-${iteration}` },
         {
           chatFlags: [
@@ -106,13 +107,14 @@ export default defineWorkflow({
         },
         {},
         async (s) => {
-          await s.session.query(
+          const result = await s.session.query(
             buildPlannerPrompt(prompt, {
               iteration,
               reviewReport: reviewReport || undefined,
             }),
           );
           s.save(s.sessionId);
+          return extractAssistantText(result, 0);
         },
       );
 
@@ -129,7 +131,33 @@ export default defineWorkflow({
         },
         {},
         async (s) => {
-          await s.session.query(buildOrchestratorPrompt(prompt));
+          await s.session.query(
+            buildOrchestratorPrompt(prompt, {
+              plannerNotes: planner.result,
+            }),
+          );
+          s.save(s.sessionId);
+        },
+      );
+
+      // ── Code Simplifier ─────────────────────────────────────────────────
+      await ctx.stage(
+        { name: `code-simplifier-${iteration}` },
+        {
+          chatFlags: [
+            "--agent",
+            "code-simplifier",
+            "--allow-dangerously-skip-permissions",
+            "--dangerously-skip-permissions",
+          ],
+        },
+        {},
+        async (s) => {
+          await s.session.query(
+            buildCodeSimplifierPrompt(prompt, {
+              plannerNotes: planner.result,
+            }),
+          );
           s.save(s.sessionId);
         },
       );

package/src/sdk/workflows/builtin/ralph/copilot/index.ts

@@ -27,6 +27,7 @@ import type { SessionEvent } from "@github/copilot-sdk";
 import {
   buildPlannerPrompt,
   buildOrchestratorPrompt,
+  buildCodeSimplifierPrompt,
   buildInfraDiscoveryPrompts,
   buildReviewPrompt,
   filterActionable,
@@ -140,6 +141,21 @@ export default defineWorkflow({
         },
       );
 
+      // ── Code Simplifier ───────────────────────────────────────────────
+      await ctx.stage(
+        { name: `code-simplifier-${iteration}` },
+        {},
+        { agent: "code-simplifier" },
+        async (s) => {
+          await s.session.send({
+            prompt: buildCodeSimplifierPrompt(userPromptText, {
+              plannerNotes: planner.result,
+            }),
+          });
+          s.save(await s.session.getMessages());
+        },
+      );
+
       // ── Infrastructure Discovery (three parallel sub-agent stages) ──
       const changeset = await captureBranchChangeset();
       const discoveryPrompts = buildInfraDiscoveryPrompts();

package/src/sdk/workflows/builtin/ralph/helpers/prompts.ts

@@ -554,6 +554,60 @@ Do NOT compress the worker's task subject, description, or persisted task
 records — those must remain self-contained and unambiguous.`);
 }
 
+// ============================================================================
+// CODE SIMPLIFIER
+// ============================================================================
+
+export interface CodeSimplifierContext {
+  /** Optional planner output (RFC markdown or absolute path to spec). */
+  plannerNotes?: string;
+}
+
+/**
+ * Build the code-simplifier prompt. Run by the `code-simplifier` sub-agent
+ * to inspect only the orchestrator's changes on the current branch and
+ * refine them for clarity, consistency, and maintainability — without
+ * altering functionality or test behaviour.
+ *
+ * @param spec - The user's original specification, used as context/fallback
+ *   when planner output is missing or ambiguous.
+ * @param context - Optional planner handoff (spec path or inline RFC markdown).
+ */
+export function buildCodeSimplifierPrompt(
+  spec: string,
+  context: CodeSimplifierContext = {},
+): string {
+  const plannerNotes = context.plannerNotes?.trim() ?? "";
+  const plannerSection =
+    plannerNotes.length > 0
+      ? `<planner_output>
+${plannerNotes}
+</planner_output>`
+      : `<planner_output>
+(empty — fall back to the Original User Specification below)
+</planner_output>`;
+
+  return withCaveman(`Inspect the orchestrator's changes on the current branch and simplify the implementation for clarity, consistency, and maintainability.
+
+## Design Input (authoritative)
+
+${plannerSection}
+
+## Original User Specification (context / fallback)
+
+<specification>
+${spec}
+</specification>
+
+## Instructions
+
+1. **Scope to branch changes only.** Run \`git diff\` and \`git status\` to identify every file the orchestrator touched. Do NOT modify files outside that set.
+2. **Simplify and refine.** Within the changed files, improve clarity, consistency, naming, and structure. Remove dead code, unnecessary comments, and redundant abstractions. Prefer simple, direct implementations.
+3. **Preserve all functionality and test behaviour.** No feature regressions, no removed tests, no altered public interfaces unless the change is strictly cosmetic (e.g. rename of an internal variable).
+4. **Keep changes minimal and focused.** No scope creep. Do not rewrite working code that is already clear. Touch only what genuinely benefits from simplification.
+5. **Verify after edits.** Run \`bun typecheck\` and \`bun lint\` from the repository root after all edits are complete. Fix any errors they surface before finishing.`);
+}
+
 // ============================================================================
 // INFRASTRUCTURE DISCOVERY
 // ============================================================================
@@ -592,7 +646,7 @@ one-line description of each.
 - **Build config**: tsconfig.json, webpack.config.*, vite.config.*, esbuild.*, rollup.config.*, Makefile, etc.
 - **Test config**: jest.config.*, vitest.config.*, playwright.config.*, .mocharc.*, pytest.ini, etc.
 - **Lint / format config**: .eslintrc.*, eslint.config.*, biome.json, .prettierrc.*, oxlint.json, etc.
-- **CI/CD workflows**: .github/workflows/*.yml, .gitlab-ci.yml, Jenkinsfile, .circleci/config.yml, etc.
+- **CI/CD workflows (REQUIRED)**: .github/workflows/*.yml, .gitlab-ci.yml, Jenkinsfile, .circleci/config.yml, azure-pipelines.yml, etc. List every workflow file separately — reviewers need full coverage.
 - **Agent config files**: CLAUDE.md, AGENTS.md, .claude/*, .github/copilot-instructions.md (these often document project commands)
 
 ## Output format
@@ -625,7 +679,12 @@ which commands to run to verify an implementation.
    executes" list.
 4. Read CLAUDE.md / AGENTS.md if present — they often document the
    canonical commands for contributors.
-5. Identify the test framework(s) in use and how to invoke them.
+5. **Audit CI workflows.** When the \`gh\` CLI is available (check via \`command -v gh\`), run:
+   - \`gh workflow list\` to enumerate active workflows.
+   - \`gh run list --limit 5\` to see the most recent runs and their statuses.
+   - \`gh workflow view <name>\` for any workflow whose name suggests it gates merges.
+   When \`gh\` is unavailable or unauthenticated, fall back to reading the workflow YAML files directly. Capture: workflow names, triggers (push / pull_request / schedule / workflow_dispatch), job names, the exact \`run:\` commands per job, and recent run statuses.
+6. Identify the test framework(s) in use and how to invoke them.
 
 ## Output format
 
@@ -644,6 +703,10 @@ which commands to run to verify an implementation.
 
 ## CI Commands (from workflow files)
 - \`<command>\` — <source file and context>
+
+## CI / GitHub Actions
+- Workflow: \`<name>\` — triggers: \`<push|pr|schedule|...>\` — jobs: \`<job names>\`
+- Recent runs (when \`gh\` available): \`<conclusion>\` on \`<branch>\` (\`<run id>\`)
 \`\`\`
 
 Be specific — include the exact invocation string (e.g. \`bun test\`, not
@@ -670,6 +733,10 @@ but HOW they are used in practice.
    order CI runs them.
 5. **Dependency install pattern**: How are dependencies installed before
    build/test (e.g. \`bun install\`, \`npm ci\`)?
+6. **CI audit patterns**: Map out the CI structure so reviewers can audit it. Identify matrix builds, job dependencies (\`needs:\`), conditional runs (\`if:\`), required checks, and which jobs gate merges. When \`gh\` is available, surface example commands a reviewer could use:
+   - \`gh run view <id> --log-failed\` — inspect a failed run.
+   - \`gh pr checks\` — see check status on the current PR.
+   - \`gh workflow view <name> --yaml\` — view the canonical workflow definition.
 
 ## Output format
 
@@ -679,7 +746,7 @@ For each pattern found, report:
 - A brief explanation of when/how it's used
 
 End with a brief trailing summary of the overall build/test workflow order
-(e.g. "install → typecheck → lint → test → build").`),
+(e.g. "install → typecheck → lint → test → build"). Also summarise CI workflow order: which workflows fire on what triggers, in what order jobs run within each.`),
   };
 }
 

package/src/sdk/workflows/builtin/ralph/opencode/index.ts

@@ -24,6 +24,7 @@ import { defineWorkflow } from "../../../index.ts";
 import {
   buildPlannerPrompt,
   buildOrchestratorPrompt,
+  buildCodeSimplifierPrompt,
   buildInfraDiscoveryPrompts,
   buildReviewPrompt,
   filterActionable,
@@ -98,7 +99,10 @@ export default defineWorkflow({
       const planner = await ctx.stage(
         { name: `planner-${iteration}` },
         {},
-        { title: `planner-${iteration}` },
+        {
+          title: `planner-${iteration}`,
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
         async (s) => {
           const result = await s.client.session.prompt({
             sessionID: s.session.id,
@@ -122,7 +126,10 @@ export default defineWorkflow({
       await ctx.stage(
         { name: `orchestrator-${iteration}` },
         {},
-        { title: `orchestrator-${iteration}` },
+        {
+          title: `orchestrator-${iteration}`,
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
         async (s) => {
           const result = await s.client.session.prompt({
             sessionID: s.session.id,
@@ -140,6 +147,31 @@ export default defineWorkflow({
         },
       );
 
+      // ── Code Simplifier ───────────────────────────────────────────────
+      await ctx.stage(
+        { name: `code-simplifier-${iteration}` },
+        {},
+        {
+          title: `code-simplifier-${iteration}`,
+          permission: [{ permission: "*", pattern: "*", action: "allow" }],
+        },
+        async (s) => {
+          const result = await s.client.session.prompt({
+            sessionID: s.session.id,
+            parts: [
+              {
+                type: "text",
+                text: buildCodeSimplifierPrompt(prompt, {
+                  plannerNotes: planner.result,
+                }),
+              },
+            ],
+            agent: "code-simplifier",
+          });
+          s.save(result.data!);
+        },
+      );
+
       // ── Infrastructure Discovery (three parallel sub-agent stages) ────
       const changeset = await captureBranchChangeset();
       const discoveryPrompts = buildInfraDiscoveryPrompts();
@@ -148,7 +180,10 @@ export default defineWorkflow({
         ctx.stage(
           { name: `infra-locate-${iteration}`, headless: true },
           {},
-          { title: `infra-locate-${iteration}` },
+          {
+            title: `infra-locate-${iteration}`,
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -162,7 +197,10 @@ export default defineWorkflow({
         ctx.stage(
           { name: `infra-analyze-${iteration}`, headless: true },
           {},
-          { title: `infra-analyze-${iteration}` },
+          {
+            title: `infra-analyze-${iteration}`,
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -176,7 +214,10 @@ export default defineWorkflow({
         ctx.stage(
           { name: `infra-patterns-${iteration}`, headless: true },
           {},
-          { title: `infra-patterns-${iteration}` },
+          {
+            title: `infra-patterns-${iteration}`,
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,
@@ -206,7 +247,10 @@ export default defineWorkflow({
         ctx.stage(
           { name },
           {},
-          { title: name },
+          {
+            title: name,
+            permission: [{ permission: "*", pattern: "*", action: "allow" }],
+          },
           async (s) => {
             const result = await s.client.session.prompt({
               sessionID: s.session.id,

package/.agents/skills/shape/SKILL.md

@@ -1,96 +0,0 @@
----
-name: shape
-description: Plan the UX and UI for a feature before writing code. Runs a structured discovery interview, then produces a design brief that guides implementation. Use during the planning phase to establish design direction, constraints, and strategy before any code is written.
-version: 2.1.1
-user-invocable: true
-argument-hint: "[feature to shape]"
----
-
-## MANDATORY PREPARATION
-
-Invoke /impeccable, which contains design principles, anti-patterns, and the **Context Gathering Protocol**. Follow the protocol before proceeding. If no design context exists yet, you MUST run /impeccable teach first.
-
----
-
-Shape the UX and UI for a feature before any code is written. This skill produces a **design brief**: a structured artifact that guides implementation through discovery, not guesswork.
-
-**Scope**: Design planning only. This skill does NOT write code. It produces the thinking that makes code good.
-
-**Output**: A design brief that can be handed off to /impeccable craft, /impeccable, or any other implementation skill.
-
-## Philosophy
-
-Most AI-generated UIs fail not because of bad code, but because of skipped thinking. They jump to "here's a card grid" without asking "what is the user trying to accomplish?" This skill inverts that: understand deeply first, so implementation is precise.
-
-## Phase 1: Discovery Interview
-
-**Do NOT write any code or make any design decisions during this phase.** Your only job is to understand the feature deeply enough to make excellent design decisions later.
-
-Ask these questions in conversation, adapting based on answers. Don't dump them all at once; have a natural dialogue. ask the user directly to clarify what you cannot infer.
-
-### Purpose & Context
-- What is this feature for? What problem does it solve?
-- Who specifically will use it? (Not "users"; be specific: role, context, frequency)
-- What does success look like? How will you know this feature is working?
-- What's the user's state of mind when they reach this feature? (Rushed? Exploring? Anxious? Focused?)
-
-### Content & Data
-- What content or data does this feature display or collect?
-- What are the realistic ranges? (Minimum, typical, maximum, e.g., 0 items, 5 items, 500 items)
-- What are the edge cases? (Empty state, error state, first-time use, power user)
-- Is any content dynamic? What changes and how often?
-
-### Design Goals
-- What's the single most important thing a user should do or understand here?
-- What should this feel like? (Fast/efficient? Calm/trustworthy? Fun/playful? Premium/refined?)
-- Are there existing patterns in the product this should be consistent with?
-- Are there specific examples (inside or outside the product) that capture what you're going for?
-
-### Constraints
-- Are there technical constraints? (Framework, performance budget, browser support)
-- Are there content constraints? (Localization, dynamic text length, user-generated content)
-- Mobile/responsive requirements?
-- Accessibility requirements beyond WCAG AA?
-
-### Anti-Goals
-- What should this NOT be? What would be a wrong direction?
-- What's the biggest risk of getting this wrong?
-
-## Phase 2: Design Brief
-
-After the interview, synthesize everything into a structured design brief. Present it to the user for confirmation before considering this skill complete.
-
-### Brief Structure
-
-**1. Feature Summary** (2-3 sentences)
-What this is, who it's for, what it needs to accomplish.
-
-**2. Primary User Action**
-The single most important thing a user should do or understand here.
-
-**3. Design Direction**
-How this should feel. What aesthetic approach fits. Reference the project's design context from `.impeccable.md` and explain how this feature should express it.
-
-**4. Layout Strategy**
-High-level spatial approach: what gets emphasis, what's secondary, how information flows. Describe the visual hierarchy and rhythm, not specific CSS.
-
-**5. Key States**
-List every state the feature needs: default, empty, loading, error, success, edge cases. For each, note what the user needs to see and feel.
-
-**6. Interaction Model**
-How users interact with this feature. What happens on click, hover, scroll? What feedback do they get? What's the flow from entry to completion?
-
-**7. Content Requirements**
-What copy, labels, empty state messages, error messages, and microcopy are needed. Note any dynamic content and its realistic ranges.
-
-**8. Recommended References**
-Based on the brief, list which impeccable reference files would be most valuable during implementation (e.g., spatial-design.md for complex layouts, motion-design.md for animated features, interaction-design.md for form-heavy features).
-
-**9. Open Questions**
-Anything unresolved that the implementer should resolve during build.
-
----
-
-ask the user directly to clarify what you cannot infer. Get explicit confirmation of the brief before finishing. If the user disagrees with any part, revisit the relevant discovery questions.
-
-Once confirmed, the brief is complete. The user can now hand it to /impeccable, or use it to guide any other implementation approach. (If the user wants the full discovery-then-build flow in one step, they should use /impeccable craft instead, which runs this skill internally.)