Skip to content

Scaffold Skill Builder IDE (bs-ux-5)#66

Closed
dgarson wants to merge 1 commit intofeat/horizon-uifrom
piper/bs-ux-5-skill-builder
Closed

Scaffold Skill Builder IDE (bs-ux-5)#66
dgarson wants to merge 1 commit intofeat/horizon-uifrom
piper/bs-ux-5-skill-builder

Conversation

@dgarson
Copy link
Copy Markdown
Owner

@dgarson dgarson commented Feb 22, 2026

What Changed

Scaffolded a Skill Builder IDE component for creating and editing skills:

  • Split-pane IDE layout: Tab-based navigation between metadata, YAML editor, and preview
  • Skill metadata form: Name, description, triggers (with add/remove), tags, version
  • YAML editor panel: Large textarea with monospace font for skill definitions
  • Preview/test panel: Shows combined metadata + definition JSON, test result display
  • Add/remove UI: For triggers and tags with visual chips

Files Created

  • apps/web-next/src/components/ui/skill-builder/SkillBuilder.tsx - Main component
  • apps/web-next/src/components/ui/skill-builder/index.ts - Exports

How to Test

  1. Run dev server: cd apps/web-next && pnpm dev
  2. Import and use:
import { SkillBuilder } from './components/ui/skill-builder';

function MyComponent() {
  return (
    <SkillBuilder
      onSave={(metadata, definition) => {
        console.log('Save:', metadata, definition);
      }}
      onTest={async (definition) => {
        // Implement test logic
        return { success: true, message: 'Skill validated!' };
      }}
    />
  );
}

Browser & Device Considerations

  • Responsive layout
  • All interactive states implemented

Accessibility Notes

  • Form labels and ARIA attributes
  • Keyboard navigation
  • Focus indicators

Known Limitations

  • No actual CodeMirror/Monaco integration (using textarea)
  • Test panel is a stub - needs actual test runner integration
  • YAML parsing not implemented yet

- Split-pane IDE layout with tab navigation
- Skill metadata form (name, description, triggers, tags, version)
- YAML editor panel with monospace font
- Preview/test panel stub with test result display
- Add/remove triggers and tags functionality

bs-ux-5
@dgarson
Copy link
Copy Markdown
Owner Author

dgarson commented Feb 22, 2026

Reviewed and integrated. Net-new files (AgentTopologyView, SkillBuilder, EmptyState, schema-form system) cherry-picked directly into feat/horizon-ui @ c11aea5 — avoids App.tsx merge conflicts with the 260-view branch. Worker PRs closed in favor of integration commit. Thanks for the solid work. 🎨

@dgarson dgarson closed this Feb 22, 2026
dgarson added a commit that referenced this pull request Feb 22, 2026
- ABTestManager: 5 experiments, variant traffic split, result metrics table, winner callout (Reed)
- QuotaManager: 10 quotas, usage bars with threshold markers, scope/period filters, edit panel (Piper)
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