prepare 0.2.27: CLI and window fixes, release tooling, deps#78
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1358966394
ℹ️ 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".
| if let window = NSApp.windows.first(where: { | ||
| $0.representedURL?.standardizedFileURL.path == normalizedPath | ||
| }) { |
There was a problem hiding this comment.
Avoid duplicating the main repo window
When the requested repo is already loaded in the SwiftUI WindowGroup but that window has not registered a representedURL yet (for example while it is still loading, or when it is showing RepoInitErrorView and never reaches RepoContentView.onAppear), this lookup misses the existing main window. Since mainWindowPath was removed, a repeated CLI/recent-repo open for the same path now falls through and creates a second host window instead of fronting the existing one; keep an explicit main-window path or assign the represented URL as soon as the root repo window is selected.
Useful? React with 👍 / 👎.
88a8a47 to
22346a6
Compare
22346a6 to
1280d7a
Compare
Groundwork for the 0.2.27 release (the version bump + release notes land on the release commit, not here).
CLI & windows
jayjay <repo>to the running instance, resolved from its bundle via a singleps -Acall (no more LaunchServices guessing between a stale/Applicationsrelease and the running build).~/.local/bin/jayjaysymlink to the current build on launch.jayjay <repo>runs while the app is running with no window (previously it just activated an empty app).ENABLE_DEBUG_DYLIB=NO) so the CLI-launched build no longer crashes on a missingJayJay.debug.dylib.UI
command failed:/Error:wrappers so command errors (e.g. resolve-conflict failures) read as plain sentences.Release tooling
just set-version <version> <build>writes every source of truth;just releaseruns acheck-versionguard that aborts on drift.dist/jayjay.rb.template(single source of truth) — fixes thezapbundle id, adds thebinarystanza (CLI on PATH for brew users) andauto_updates.Deps
tree-sitter0.26.9,similar3.1.1,tree-sitter-swift0.7.3.Verified:
cargo build/clippy/test --workspace,just build,just test-app,just lintall green.