Skip to content

chore(copilot): sync load_slide_layout into the tool catalog - #7167

Merged
j15z merged 1 commit into
stagingfrom
chore/sync-load-slide-layout-catalog
Aug 27, 2026
Merged

chore(copilot): sync load_slide_layout into the tool catalog#7167
j15z merged 1 commit into
stagingfrom
chore/sync-load-slide-layout-catalog

Conversation

@j15z

@j15z j15z commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

Type of Change

  • Chore

Testing

  • bunx vitest run lib/copilot/tools/tool-display.test.ts lib/copilot/tools/client/hidden-tools.test.ts, bun run type-check

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 27, 2026 5:47pm

Request Review

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 4 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

The file agent's slide-layout loader (Go-executed, never run by Sim) was
added to the mothership catalog but never synced here, so its transcript
row fell back to a humanized title. Sync the catalog entry, hide the row
like load_skill, and keep a fallback title for history.
@j15z
j15z changed the base branch from feat/let-mothership-edit-views to staging August 27, 2026 17:47
@j15z
j15z force-pushed the chore/sync-load-slide-layout-catalog branch from dff9bd6 to 4a52f03 Compare August 27, 2026 17:47
@greptile-apps

greptile-apps Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR synchronizes the Go-executed load_slide_layout tool into Sim’s generated catalog and runtime schemas, then treats its reference-loading transcript event consistently with load_skill.

  • Registers the tool’s required name parameter and Go routing metadata.
  • Hides the loader from transcript rows while retaining a fallback display title.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/sim/lib/copilot/generated/tool-catalog-v1.ts Adds the generated LoadSlideLayout catalog entry and registration using the established ToolCatalogEntry output shape.
apps/sim/lib/copilot/generated/tool-schemas-v1.ts Adds the corresponding runtime parameter schema with a required string layout name.
apps/sim/lib/copilot/tools/client/hidden-tools.ts Classifies the slide-layout reference loader as hidden consistently with the analogous skill loader.
apps/sim/lib/copilot/tools/tool-display.ts Adds a fallback human-readable title for slide-layout loading history.

Reviews (2): Last reviewed commit: "chore(copilot): sync load_slide_layout i..." | Re-trigger Greptile

Comment thread apps/sim/lib/copilot/generated/tool-catalog-v1.ts
@j15z

j15z commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@j15z
j15z merged commit 474c105 into staging Aug 27, 2026
30 of 31 checks passed
@j15z
j15z deleted the chore/sync-load-slide-layout-catalog branch August 27, 2026 22:27
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