Skip to content

Onboarding improvements - #263

Merged
alecmocatta merged 2 commits into
mainfrom
am
Jul 3, 2026
Merged

Onboarding improvements#263
alecmocatta merged 2 commits into
mainfrom
am

Conversation

@alecmocatta

Copy link
Copy Markdown
Contributor
  • Open most recent markdown file on empty workspace
  • Auto-launch preferred agent

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
flashtype-website 1df50ee Commit Preview URL

Branch Preview URL
Jul 03 2026, 09:02 AM

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1df50ee889

ℹ️ 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".

Comment thread electron/agent-status.mjs
: candidates[0];
return {
agents,
autoLaunchAgent: tier.autoLaunch ? preferredAgent : null,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Require supported versions before auto-launch

When a paid/free agent reports auth successfully but checkAgentVersionSupport returned false, this still returns it as autoLaunchAgent because the auto-launch tiers do not consider supportedVersion. In that scenario, such as an outdated paid Claude install with a supported Codex install also available, the renderer opens a terminal that immediately fails the version preflight instead of choosing a runnable agent; gate auto-launch candidates on status.supportedVersion or rank unsupported agents below supported ones.

Useful? React with 👍 / 👎.

};
}, [
extensionMap,
handleAddView,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep the one-shot preference probe alive across file opens

Including handleAddView here makes the effect tear down whenever the active central file changes, since that callback depends on activeCentralEntry. If the recent-markdown fallback or a pending launch file opens while getPreferredAgent() is still probing, the cleanup sets cancelled = true; the rerun then exits because agentPreferenceAttemptedRef was already set, so the preferred CTA/auto-launch is skipped for the rest of the workspace. Use a stable ref for adding the view or only mark the attempt after the probe finishes.

Useful? React with 👍 / 👎.

@alecmocatta
alecmocatta enabled auto-merge July 3, 2026 09:12
@alecmocatta
alecmocatta merged commit f084430 into main Jul 3, 2026
4 checks passed
@alecmocatta
alecmocatta deleted the am branch July 3, 2026 09:16
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