Skip to content

docs(skills): require GitHub API fallback for issue comments - #171

Merged
egdev6 merged 1 commit into
mainfrom
docs/issue-comments-fallback
May 18, 2026
Merged

egdev6 merged 1 commit into
mainfrom
docs/issue-comments-fallback

Conversation

@Gabo-Dev

Copy link
Copy Markdown
Collaborator

Summary

  • Add a GitHub API fallback to the component-contributor and component-spec-proposer skills when the rendered issue page does not expose comments.
  • Prevent the intake flow from missing validated specs that live in issue comment threads.

Closes #170

Type of change

  • 🧩 New component
  • 🐛 Bug fix
  • 🎨 Design tokens
  • ♿ Accessibility
  • 🏗️ Infrastructure
  • 📚 Documentation

Component checklist (skip if not applicable)

  • Follows the 5-file structure (types.ts, use*.ts, Component.tsx, index.ts, *.stories.tsx)
  • CVA variants defined in types.ts, not inline
  • No hardcoded colors — uses tokens from theme.css
  • No any types — TypeScript strict
  • ARIA attributes present and correct
  • Keyboard navigable (Tab, Enter, Escape where applicable)
  • Dark mode works correctly
  • Storybook stories cover: default, variants, states (hover, focus, disabled)
  • Tests added or updated

How to test

  1. Open a GitHub issue that stores the validated spec in comments.
  2. Verify the skills now instruct the agent to fetch comments through the GitHub API or gh api if the rendered page does not expose them.
  3. Confirm the repository still passes pnpm run typecheck via the pre-commit hook on commit.

Notes for reviewer

This change is a harness/docs correction only. No runtime component behavior changed.

@Gabo-Dev
Gabo-Dev requested a review from egdev6 as a code owner May 18, 2026 12:02
@Gabo-Dev Gabo-Dev added the type:docs Documentation changes label May 18, 2026
@egdev6
egdev6 merged commit 392ac90 into main May 18, 2026
6 checks passed
@egdev6
egdev6 deleted the docs/issue-comments-fallback branch May 18, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:docs Documentation changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UPDATE] fetch issue comments via GitHub API fallback in component skills

2 participants