Skip to content

fix(docs): update simstudio.ai URLs to sim.ai in SSO docs#4257

Merged
waleedlatif1 merged 5 commits intostagingfrom
waleedlatif1/docs-simstudio-to-sim
Apr 22, 2026
Merged

fix(docs): update simstudio.ai URLs to sim.ai in SSO docs#4257
waleedlatif1 merged 5 commits intostagingfrom
waleedlatif1/docs-simstudio-to-sim

Conversation

@waleedlatif1
Copy link
Copy Markdown
Collaborator

Summary

  • Replace all simstudio.ai domain references in the SSO docs with sim.ai (11 occurrences in apps/docs/content/docs/en/enterprise/sso.mdx)

Type of Change

  • Bug fix

Testing

Tested manually

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
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 22, 2026 4:20am

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 22, 2026

PR Summary

Medium Risk
Mostly documentation URL updates, but the UI changes introduce duplicate handleStopGeneration/handleCopilotStopGeneration declarations in React components, which is likely to break TypeScript builds and alters stop-generation error handling.

Overview
Updates the Enterprise SSO documentation to replace simstudio.ai with sim.ai across callback URLs, entity IDs, and example flows.

In the workspace UI (home.tsx and workflow panel.tsx), adds new stop-generation callbacks that remove the previous void ...catch handling, but currently leaves duplicate handleStopGeneration/handleCopilotStopGeneration definitions that will likely cause a compile-time redeclaration error.

Reviewed by Cursor Bugbot for commit 77c5dbd. Configure here.

@waleedlatif1 waleedlatif1 force-pushed the waleedlatif1/docs-simstudio-to-sim branch from 77c5dbd to 382f7fb Compare April 22, 2026 03:44
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 22, 2026

Greptile Summary

This PR updates simstudio.aisim.ai in SSO documentation (clean, correct) and adds captureEvent analytics tracking to stop-generation handlers in home.tsx and panel.tsx. However, both component files now contain duplicate const declarations for handleStopGeneration and handleCopilotStopGeneration in the same function scope — TypeScript will reject these as "Cannot redeclare block-scoped variable," breaking the build.

  • home.tsx lines 204 and 212 both declare const handleStopGeneration; the PR should update the existing declaration instead of adding a second one.
  • panel.tsx lines 396 and 404 both declare const handleCopilotStopGeneration with identical bodies; lines 404–410 must be removed.

Confidence Score: 2/5

Not safe to merge — duplicate const declarations in two component files will fail TypeScript compilation.

Two P0 build-breaking duplicate variable declarations were introduced alongside an otherwise clean documentation fix. These must be resolved before merging.

apps/sim/app/workspace/[workspaceId]/home/home.tsx and apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx both have duplicate const declarations that need to be fixed.

Important Files Changed

Filename Overview
apps/docs/content/docs/en/enterprise/sso.mdx 11 occurrences of simstudio.ai replaced with sim.ai in callback URLs, redirect URIs, and documentation text — straightforward and correct.
apps/sim/app/workspace/[workspaceId]/home/home.tsx Adds a second const handleStopGeneration in the same scope as the existing one (line 204), creating a TypeScript duplicate-declaration error that breaks compilation.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx Adds a second identical const handleCopilotStopGeneration in the same scope as the existing one (line 396), creating a TypeScript duplicate-declaration error that breaks compilation.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User clicks Stop] --> B{Which view?}
    B -->|mothership| C[handleStopGeneration]
    B -->|copilot| D[handleCopilotStopGeneration]
    C --> E[captureEvent: task_generation_aborted
view: mothership]
    D --> F[captureEvent: task_generation_aborted
view: copilot]
    E --> G[stopGeneration]
    F --> H[copilotStopGeneration]
    style C fill:#f99,stroke:#f00
    style D fill:#f99,stroke:#f00
Loading

Comments Outside Diff (2)

  1. apps/sim/app/workspace/[workspaceId]/home/home.tsx, line 212-218 (link)

    P0 Duplicate const declaration breaks TypeScript compilation

    handleStopGeneration is already declared at line 204 with void stopGeneration().catch(() => {}). Adding a second const handleStopGeneration in the same scope is a TypeScript error ("Cannot redeclare block-scoped variable") and will break the build. The cleanest fix is to delete lines 212–218 and instead update the existing declaration at line 204 to add the captureEvent call before void stopGeneration().catch(() => {}).

  2. apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx, line 404-410 (link)

    P0 Duplicate const declaration breaks TypeScript compilation

    handleCopilotStopGeneration is already declared at line 396 with an identical body. Both definitions are in the same function scope, producing a TypeScript "Cannot redeclare block-scoped variable" error that will fail the build. Lines 404–410 should be removed entirely; the existing declaration at line 396 is sufficient.

Reviews (1): Last reviewed commit: "fix(docs): update simstudio.ai URLs to s..." | Re-trigger Greptile

@waleedlatif1 waleedlatif1 merged commit 7941dcd into staging Apr 22, 2026
8 of 9 checks passed
@waleedlatif1 waleedlatif1 deleted the waleedlatif1/docs-simstudio-to-sim branch April 22, 2026 05:14
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