Skip to content
This repository was archived by the owner on Jun 23, 2025. It is now read-only.

Implement lint, test, docs, coverage, and attempt to debug some flakey stdio things - #10

Open
andrewxhill wants to merge 7 commits into
mainfrom
andrew/qa-hits
Open

Implement lint, test, docs, coverage, and attempt to debug some flakey stdio things#10
andrewxhill wants to merge 7 commits into
mainfrom
andrew/qa-hits

Conversation

@andrewxhill

Copy link
Copy Markdown
Member

@mzkrasner look closely at any changes to the actual code. the main thing i was doing was trying to chase down some kinda stdio start stop thing i was seeing.
but i gave cursor a laundry list of qa things to hit along the way.

Key Changes:

  • Code Quality: Added ESLint, Prettier and pre-commit hooks
  • Testing: Implemented unit and integration tests with 85%+ coverage
  • Documentation: Added TSDoc with >90% coverage
  • CI/CD: Set up GitHub Actions workflow for all quality checks
  • Reliability: Fixed flaky stdio issues with error listeners, AbortSignal handling, and keep-alive pings

The project now meets all success criteria defined in the development plan, with all scripts (lint, docs, test, coverage) passing both locally and in CI.

Signed-off-by: Devrel Intern <intern@recalllabs.ai>
Signed-off-by: Devrel Intern <intern@recalllabs.ai>
@andrewxhill
andrewxhill requested a review from mzkrasner April 22, 2025 02:09
Signed-off-by: Devrel Intern <intern@recalllabs.ai>
Signed-off-by: Devrel Intern <intern@recalllabs.ai>
Signed-off-by: Devrel Intern <intern@recalllabs.ai>
Signed-off-by: Devrel Intern <intern@recalllabs.ai>
Signed-off-by: Devrel Intern <intern@recalllabs.ai>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants