Skip to content

Surface dock tools, app URLs, and project IDs in presenter output#257

Merged
jeremy merged 1 commit intomainfrom
pr/presenter-affordances
Mar 11, 2026
Merged

Surface dock tools, app URLs, and project IDs in presenter output#257
jeremy merged 1 commit intomainfrom
pr/presenter-affordances

Conversation

@jeremy
Copy link
Copy Markdown
Member

@jeremy jeremy commented Mar 11, 2026

Summary

  • surface project IDs more prominently in presenter output
  • add dock rendering for enabled project tools
  • include app URLs in project, todo, todolist, and comment schemas

Testing

  • bin/ci

Summary by cubic

Adds a dock section to Project details and surfaces plain-text app_url across schemas to make presenter output easier to navigate. Also shows Project IDs at normal weight for quick copy/paste.

  • New Features
    • Project dock: new dock field + formatter lists enabled tools as "Title (name, ID: xxx)" in the Project detail view.
    • app_url added to Project, Todo, Todolist, and Comment schemas and shown in Metadata for terminals without link support.
    • Project id is no longer muted and is shown in Metadata for easier copying.

Written for commit cad0833. Summary will update on new commits.

Copilot AI review requested due to automatic review settings March 11, 2026 03:50
@jeremy jeremy requested a review from a team as a code owner March 11, 2026 03:50
@github-actions github-actions bot added the output Output formatting and presentation label Mar 11, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 5 files

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enhances the presenter’s human-readable output by surfacing Basecamp “app URLs”, making project IDs more prominent, and adding a rendered view of enabled project dock tools.

Changes:

  • Add app_url to the presenter schemas for projects, todos, todolists, and comments (and show it in detail “Metadata” sections).
  • Add a dock field to the project schema and introduce a dock field formatter.
  • Adjust project schema metadata presentation (project id no longer muted; dock gets its own detail section).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
internal/presenter/schemas/todolist.yaml Adds app_url field and includes it in the detail metadata section.
internal/presenter/schemas/todo.yaml Adds app_url field and includes it in the detail metadata section.
internal/presenter/schemas/project.yaml Adds dock and app_url fields; adds a “Dock” section; makes id more prominent.
internal/presenter/schemas/comment.yaml Adds app_url field and includes it in the detail metadata section.
internal/presenter/format.go Adds dock format support and a new formatDock formatter.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ddc9d119a8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- Add dock section to project detail view with new "dock" format
  that renders enabled tools as "Title (name, ID: xxx)"
- Add app_url field to project, todo, todolist, and comment schemas
  so the URL appears in plain-text output for terminals without OSC 8
- Remove muted emphasis from project id so it renders at normal weight
@jeremy jeremy force-pushed the pr/presenter-affordances branch from ddc9d11 to cad0833 Compare March 11, 2026 04:29
@github-actions github-actions bot added the tests Tests (unit and e2e) label Mar 11, 2026
@jeremy jeremy merged commit b270db3 into main Mar 11, 2026
25 checks passed
@jeremy jeremy deleted the pr/presenter-affordances branch March 11, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

output Output formatting and presentation tests Tests (unit and e2e)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants