Skip to content

fix(docs): document You.com tools and expand shared output spreads - #8592

Merged
waleedlatif1 merged 2 commits into
stagingfrom
fix/youcom-op-descriptions
Oct 3, 2026
Merged

waleedlatif1 merged 2 commits into
stagingfrom
fix/youcom-op-descriptions

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • The You.com block declared its operation ids and tools.access through constants (id: SEARCH). scripts/generate-docs.ts parses these from source with regexes, so they read as empty: the docs page had no Actions section and all 8 catalog operations shipped with blank descriptions (flagged on v0.9.12: Power BI integration, library articles, chat fork speedups #8589). Switched to string literals like every other block, and added the rule to the add-block skill
  • expandSpreadConsts is documented to inline ...sharedConst spreads in both params: and outputs:, but it only ran on params. Any tool that spreads shared outputs lost those rows from its docs table. It now runs on outputs too, including inherited base tools
  • Moved You.com's shared research outputs into spreadable groups (RESEARCH_SOURCE_OUTPUTS, RESEARCH_ANSWER_OUTPUTS), so Research, Get Research Task, and Finance Research document content, contentType, sources, and warnings
  • The generator fix also corrects 29 output tables across 12 other integrations (ashby, github, jira, stripe, salesforce, …). Every one of them now exactly matches the runtime tool metadata; previously some showed the whole block's outputs or dropped spread rows

Type of Change

  • Bug fix

Testing

  • Added a regression test in scripts/generate-docs.test.ts for a shared output spread; it fails on the old generator and passes now
  • Compared every changed output table against apps/sim/tools/generated/tool-outputs.ts: 29 changed tables now match it exactly, none regressed
  • bun run test (34,649 passed), bun run lint, bun run type-check, check:audits (58/58), docs-manifest:check, check-block-registry.ts origin/staging

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • 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 Oct 2, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Oct 2, 2026 11:43pm UTC

Request Review

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

Copy link
Copy Markdown
Contributor

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

@greptile-apps

greptile-apps Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Updates documentation and tool configuration for integration catalog.

The PR appears safe to merge, with non-blocking documentation regressions worth correcting.

Findings

  1. P2 Output details disappear ▶
  2. P2 Completion condition is undocumented ▶

Summary

This PR makes You.com operation IDs discoverable to the documentation generator and expands shared research outputs in generated integration documentation.

  • The You.com Actions section and research output rows are restored.
  • The broader regeneration drops documented Coda output details and operation-specific descriptions; Get Research Task also loses a useful completion qualifier.

Reviews (2) · Last reviewed commit: "fix(docs): expand shared output spreads ..."

Comment thread apps/docs/content/docs/integrations/youcom.mdx
@waleedlatif1
waleedlatif1 force-pushed the fix/youcom-op-descriptions branch from 1983744 to 1201027 Compare October 2, 2026 23:40
@waleedlatif1 waleedlatif1 changed the title fix(youcom): use literal operation ids so docs and catalog pick up tools fix(docs): document You.com tools and expand shared output spreads Oct 2, 2026
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

Comment thread scripts/generate-docs.ts
Comment thread apps/sim/tools/youcom/get_research_task.ts

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 23 files

Confidence score: 5/5

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

Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 52878d1 into staging Oct 3, 2026
33 of 34 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/youcom-op-descriptions branch October 3, 2026 04:13

This branch was successfully deployed

1 active deployment
Preview — 12010270 Deployed Oct 2, 2026 by vercel[bot]
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