Skip to content

chore(tui): run terminal tests in CI and clarify group guide - #39

Merged
yswcyswc merged 3 commits into
mainfrom
cathyw/group-ci-docs
Oct 9, 2026
Merged

yswcyswc merged 3 commits into
mainfrom
cathyw/group-ci-docs

Conversation

@yswcyswc

@yswcyswc yswcyswc commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Issue for this PR

Related to #12. Follows #25 and #29.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Adds @opencode-ai/tui#test to Turbo so the existing CI test job includes the terminal tests, including the repository-map unit, integration, and E2E coverage. Previously this package was absent from the configured test tasks.

Organizes UserGuide.md section into usage, numbered user checks, and automated tests, with focused commands and an accurate explanation of session prioritization.

How did you verify your code works?

  • Validated the Turbo configuration and confirmed other teammates' guide sections are unchanged.
  • git diff --check passed. Two files changed: 33 additions and 3 deletions.

Screenshots / recordings

Not applicable: CI configuration and documentation only.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@ItsJayYip

Copy link
Copy Markdown
Contributor

I think these changes are pretty helpful, especially adding the terminal tests to the CI pipeline since we already have a good amount of test coverage for the repository map. It makes sense to have those tests run automatically rather than relying on someone to run them locally every time. I also like how you reorganized the UserGuide.md section to separate the usage instructions, manual checks, and automated tests. It makes the documentation easier to follow, especially for someone who hasn't worked with the feature before. I think this is a good improvement to both the testing workflow and documentation that you mentioned to our team.

@yswcyswc
yswcyswc merged commit ed13f18 into main Oct 9, 2026
7 of 8 checks passed
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.

2 participants