Skip to content

Conversation

@sreya
Copy link
Contributor

@sreya sreya commented Nov 13, 2025

  • Rebuilt AGENTS.md from scratch (390 lines, AI agent-focused)

    • Added practical code patterns and workflows
    • Removed outdated historical content
    • Updated architecture diagram and metrics
  • Removed unused SgrParser class from lib/ghostty.ts (-175 lines)

    • Kept KeyEncoder (actively used by InputHandler)
    • Cleaned up unused type imports and exports
  • Deleted outdated documentation files

    • docs/API.md (1,046 lines)
    • roadmap.md (921 lines)
    • run-demo.sh (83 lines)
  • Updated references in README.md and lib/index.ts

Net reduction: 2,402 lines (~89% cleanup)
All 88 tests passing ✅

- Rebuilt AGENTS.md from scratch (390 lines, AI agent-focused)
  - Added practical code patterns and workflows
  - Removed outdated historical content
  - Updated architecture diagram and metrics

- Removed unused SgrParser class from lib/ghostty.ts (-175 lines)
  - Kept KeyEncoder (actively used by InputHandler)
  - Cleaned up unused type imports and exports

- Deleted outdated documentation files
  - docs/API.md (1,046 lines)
  - roadmap.md (921 lines)
  - run-demo.sh (83 lines)

- Updated references in README.md and lib/index.ts

Net reduction: 2,402 lines (~89% cleanup)
All 88 tests passing ✅
@sreya sreya force-pushed the cleanup-agents-docs branch 2 times, most recently from fdeca76 to d9d43c3 Compare November 13, 2025 11:44
- Removed SgrParser class from lib/ghostty.ts (163 lines)
- Removed createSgrParser() method from Ghostty class
- Cleaned up unused imports (SgrAttribute, SgrAttributeTag, RGBColor)
- Updated lib/index.ts exports to remove SgrParser
- lib/ghostty.ts: 708 → 545 lines (-163 lines, -23%)

All tests passing ✅
@sreya sreya force-pushed the cleanup-agents-docs branch 2 times, most recently from 583075c to f02c49e Compare November 13, 2025 11:49
- Added "Before Committing" section with all CI checks
- Updated test count from ~129 to 95 (current count)
- Emphasized importance of running fmt, lint, typecheck, test, build
@sreya sreya force-pushed the cleanup-agents-docs branch from f02c49e to 6a26bd1 Compare November 13, 2025 11:49
Auto-formatted with 'bun run fmt:fix' to pass CI checks
@sreya sreya merged commit 20c9d29 into main Nov 13, 2025
5 checks passed
sreya added a commit that referenced this pull request Nov 13, 2025
* Cleanup: Rebuild AGENTS.md and remove legacy docs

- Rebuilt AGENTS.md from scratch (390 lines, AI agent-focused)
  - Added practical code patterns and workflows
  - Removed outdated historical content
  - Updated architecture diagram and metrics

- Removed unused SgrParser class from lib/ghostty.ts (-175 lines)
  - Kept KeyEncoder (actively used by InputHandler)
  - Cleaned up unused type imports and exports

- Deleted outdated documentation files
  - docs/API.md (1,046 lines)
  - roadmap.md (921 lines)
  - run-demo.sh (83 lines)

- Updated references in README.md and lib/index.ts

Net reduction: 2,402 lines (~89% cleanup)
All 88 tests passing ✅

* Remove unused SgrParser class

- Removed SgrParser class from lib/ghostty.ts (163 lines)
- Removed createSgrParser() method from Ghostty class
- Cleaned up unused imports (SgrAttribute, SgrAttributeTag, RGBColor)
- Updated lib/index.ts exports to remove SgrParser
- lib/ghostty.ts: 708 → 545 lines (-163 lines, -23%)

All tests passing ✅

* docs: update AGENTS.md with pre-commit checks

- Added "Before Committing" section with all CI checks
- Updated test count from ~129 to 95 (current count)
- Emphasized importance of running fmt, lint, typecheck, test, build

* fix: apply prettier formatting to all files

Auto-formatted with 'bun run fmt:fix' to pass CI checks
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