GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix(mship, desktop): fix bugs by Sg312 · Pull Request #6572 · simstudioai/sim

Conversation

@Sg312

@Sg312 Sg312 commented Aug 11, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Summary

Fix mship and desktop app bugs
Companion: https://github.com/simstudioai/mothership/pull/433

Type of Change

  • Bug fix

Testing

Manual

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)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@Sg312
fix bugs
fd18729
@vercel

vercel Bot commented Aug 11, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

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 12, 2026 12:12am

Request Review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@greptile-apps

greptile-apps Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR revises desktop release streams and update handling, fixes Copilot abort ordering, and adjusts Mothership interaction rendering. It also removes desktop folder-management controls while persistent filesystem grants remain active.

  • Renames prerelease streams from alpha/beta to dev/staging while preserving legacy compatibility.
  • Adds origin-aware desktop update feeds, stricter manifest validation, timeouts, and one-action download/install behavior.
  • Writes Copilot’s explicit abort marker before terminating the local stream.
  • Collapses superseded credential prompts and streamlines single-select question interaction.
  • Removes the Desktop settings interface for local-folder grants.

Confidence Score: 3/5

The PR should not merge until users retain an in-app way to inspect and revoke persistent local-folder access.

Existing filesystem grants survive restarts and remain available to Copilot tools, while this change removes the sole settings UI that listed and revoked them; the new route test also violates the repository’s absolute-import convention.

Files Needing Attention: apps/sim/app/workspace/[workspaceId]/settings/components/desktop/desktop.tsx; apps/sim/app/api/desktop/update/latest-mac.yml/route.test.ts

Security Review

The Desktop settings change removes the only in-app revocation path for persistent local-folder grants. Existing grants are restored at startup and remain usable by Copilot filesystem tools, leaving users unable to inspect or revoke that access from the application.

Important Files Changed

Filename Overview
apps/desktop/src/main/updater.ts Reworks update-channel resolution, feed probing, candidate validation, timeouts, and explicit download-to-install behavior; no confirmed blocking updater defect remains.
apps/sim/app/api/desktop/update/latest-mac.yml/route.ts Selects update streams from the request hostname, distinguishes authoritative no-release responses, and validates manifest versions against release tags.
apps/sim/app/api/copilot/chat/abort/route.ts Moves the bounded Go abort-marker request before local SSE teardown to prevent deliberate stops from being misclassified.
apps/sim/app/workspace/[workspaceId]/settings/components/desktop/desktop.tsx Simplifies desktop settings but removes the only UI for inspecting and revoking persistent local filesystem grants.
apps/sim/app/api/desktop/update/latest-mac.yml/route.test.ts Adds update-feed coverage but introduces a relative import contrary to the apps/sim absolute-import rule.
apps/sim/app/workspace/[workspaceId]/home/components/mothership-chat/mothership-chat.tsx Marks superseded credential interactions as abandoned so historical prompts render as recaps instead of active forms.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Shell[Desktop shell] --> Feed[Environment update feed]
  Feed --> Release[Matching dev, staging, or stable release]
  Release --> Updater[Desktop updater]
  Updater --> Download[Download update]
  Download --> Install[Install and relaunch]
  User[User grants local folder] --> Store[Persistent encrypted grant store]
  Store --> Restore[Restore grant at startup]
  Restore --> Tools[Copilot filesystem tools]
  Settings[Desktop settings] -. no revoke control .-> Store
Loading

Reviews (1): Last reviewed commit: "fix bugs" | Re-trigger Greptile

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot]
greptile-apps Bot reviewed Aug 11, 2026
View reviewed changes
Comment thread apps/sim/app/workspace/[workspaceId]/settings/components/desktop/desktop.tsx Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread apps/sim/app/api/desktop/update/latest-mac.yml/route.test.ts Resolved

Uh oh!

There was an error while loading. Please reload this page.

@vercel
vercel Bot deployed to Preview August 11, 2026 23:49
View deployment
@Sg312
fix tests
48692c8
@vercel
vercel Bot temporarily deployed to Preview August 11, 2026 23:54 Inactive
@github-actions github-actions Bot added the requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep label Aug 11, 2026
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

  • ❌ simstudioai/mothership#433 — OPEN, not merged (targets staging) — fix(mship): fix bugs
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@Sg312
fix(tests): stabilize archive and client boundary coverage
2f90889
@vercel
vercel Bot temporarily deployed to Preview August 12, 2026 00:12 Inactive
@Sg312
Sg312 merged commit cb28090 into staging Aug 12, 2026
28 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1
waleedlatif1 deleted the improvement-desktop branch August 12, 2026 00:18
@waleedlatif1 waleedlatif1 mentioned this pull request Aug 12, 2026
v0.8.0: desktop app, workflow UI, v2 endpoints #6606
Merged
@reddynitish reddynitish mentioned this pull request Aug 27, 2026
Pre-existing test failure on staging: file/manage secret-bearing archive provenance #6574
Open

This branch was previously deployed

1 inactive deployment
Preview — 2f908894 Deployed Aug 12, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@Sg312
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.