Skip to content

Add a custom "/scotty" command to OpenCode CLI that prints out an ASCII art of Scotty - #208

Closed
zrusekko-cmu wants to merge 3 commits into
CMU-313:mainfrom
zrusekko-cmu:recitation-03/add-scotty-command
Closed

zrusekko-cmu wants to merge 3 commits into
CMU-313:mainfrom
zrusekko-cmu:recitation-03/add-scotty-command

Conversation

@zrusekko-cmu

@zrusekko-cmu zrusekko-cmu commented Sep 15, 2026 •

Copy link
Copy Markdown

Issue for this PR

Closes #2

Type of change

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

What does this PR do?

This PR adds a custom /scotty command to the OpenCode CLI. Running the command opens a dialog displaying an ASCII art of Scotty. The command is registered in the TUI command list and uses a new dialog-scotty component to display the artwork. The changes are limited to the files needed for the new command and do not modify any unrelated functionality.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

AI Usage

I used AI assistance during the implementation of this feature.

How did you verify your code works?

I ran the OpenCode CLI locally and tested the /scotty command to verify that the command is available and that the Scotty dialog opens and displays correctly. I also ran bun test in the TUI package as well as bun lint for the specific files changed (i.e., packages/tui/src/component/dialog-scotty.tsx and packages/tui/src/app.tsx), both passing successfully.

Output of running bun test in the TUI package:
Screenshot 2026-09-14 at 10 15 36 PM

Output of running bun lint for the specific files changed:
Screenshot 2026-09-14 at 10 13 11 PM

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

/scotty command now available in OpenCode:
Screenshot 2026-09-14 at 10 12 11 PM

Scotty ASCII art displayed after running /scotty command in OpenCode:
Screenshot 2026-09-14 at 10 12 24 PM

Checklist

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

If you do not follow this template your PR will be automatically rejected.

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.

P1B: Refactor (script/translate-app.ts:247): Function with high complexity (count = 23): main

1 participant