Skip to content

Conversation

@caocuong2404
Copy link

@caocuong2404 caocuong2404 commented Jan 16, 2026

Summary

This PR adds Amp (Sourcegraph's AI coding agent) support and reduces repository duplication by replacing duplicate directories with symlinks.

Changes

Amp Compatibility

  • Fix Python 3.10 f-string syntax in design_system.py - backslashes in f-strings require Python 3.12+
  • Convert SKILL.md from CRLF to LF - fixes YAML frontmatter parsing error in Amp
  • Add Amp installation docs in README with CLI and manual instructions
  • Add .agents symlink pointing to .claude/skills/ui-ux-pro-max for Amp workspace installs

Symlink Optimization

  • Replace 12 duplicate directories in cli/assets/ with symlinks to root counterparts
  • Remove 153 duplicate files (~14.4K lines of code)
  • Reduces maintenance burden - edit once, apply everywhere
Directory Symlink Target
.agent ../../.agent
.claude ../../.claude
.codex ../../.codex
.cursor ../../.cursor
.gemini ../../.gemini
.github ../../.github
.kiro ../../.kiro
.qoder ../../.qoder
.roo ../../.roo
.shared ../../.shared
.trae ../../.trae
.windsurf ../../.windsurf

Testing

  • Verified CLI init --ai claude installs correctly
  • Verified CLI init --ai amp installs correctly
  • Verified skill loads without errors in Amp

Fixes

Fixes skill loading error Missing YAML frontmatter in Amp caused by CRLF line endings.

@caocuong2404 caocuong2404 force-pushed the fix/amp-compatibility branch from 30f7d52 to af7203e Compare January 16, 2026 08:41
- Fix f-string backslash syntax in design_system.py (Python <3.12 compat)
- Convert SKILL.md from CRLF to LF (fixes YAML frontmatter parsing in Amp)
- Add Amp installation and usage documentation in README

Fixes: skill loading error 'Missing YAML frontmatter' in Amp
@caocuong2404 caocuong2404 force-pushed the fix/amp-compatibility branch from af7203e to 5094cc5 Compare January 16, 2026 08:43
Replace cli/assets/{.codex,.gemini,.github,.kiro,.roo,.shared,.trae,.windsurf}
with symlinks to their root counterparts.

This reduces 109 duplicate files and ~10.7K lines of code while
maintaining the same behavior for the CLI installer.

Affected directories:
- .codex -> ../../.codex
- .gemini -> ../../.gemini
- .github -> ../../.github
- .kiro -> ../../.kiro
- .roo -> ../../.roo
- .shared -> ../../.shared
- .trae -> ../../.trae
- .windsurf -> ../../.windsurf
@caocuong2404 caocuong2404 force-pushed the fix/amp-compatibility branch from 3f8bfb0 to 4ff1375 Compare January 16, 2026 09:06
@caocuong2404 caocuong2404 changed the title fix: Amp compatibility - Python 3.10 f-string and CRLF line endings feat: Amp compatibility + symlink optimization Jan 16, 2026
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