Skip to content

Make export/import unconditional and drop its setting - #268

Merged
scgopi merged 1 commit into
mainfrom
remove-share-flag
Sep 3, 2026
Merged

Make export/import unconditional and drop its setting#268
scgopi merged 1 commit into
mainfrom
remove-share-flag

Conversation

@scgopi

@scgopi scgopi commented Sep 3, 2026

Copy link
Copy Markdown
Owner

What

Export/import of loops is now always available. GraphcodeSettings.sharesLoops is gone, along with the "Export and import loops" toggle in Settings.

Why

The switch has defaulted to on since 0.1.40. Every surface behind it is a right-click menu item that does nothing until deliberately clicked — nothing runs, nothing is spent — so the toggle only offered a way to hide working verbs from yourself.

Changes

Area Change
GraphcodeSettings property, init parameter and decode migration removed
SettingsView toggle + its caption removed
Canvas / overview / sidebar menus five if sharesLoops gates unwrapped
AppSidebarView stored flag dropped; the segfault note it carried moved to offersCodespaces, which needs it for the same reason
graphcode-cli requireLoopSharing() and its dispatch removed
SessionBriefing the "stay off until enabled in Settings" caveat dropped from the export/import bullet
Tests three assertions repointed to other settings; briefing test now checks the node import verb

A settings file that still records "sharesLoops": false is ignored — that is what enabling by default means here.

Verification

Check Result
xcodebuild -scheme graphcode build ✅ 0 errors
-scheme graphcode-cli / -scheme graphcoded ✅ 0 errors
Tests ✅ 1537 tests in 160 suites passed
swiftlint lint ✅ 0 errors
swift format lint --strict ✅ clean

🤖 Generated with Claude Code

https://claude.ai/code/session_013EUE64JZc1QUodfL8LnPkK

The switch was on by default since 0.1.40 and every surface behind it is a
menu item that does nothing until clicked, so the toggle only offered a way
to hide working verbs. Removes `GraphcodeSettings.sharesLoops`, its Settings
toggle, the five menu gates, and the CLI's refusal; a settings file that
still says `sharesLoops: false` is now ignored.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013EUE64JZc1QUodfL8LnPkK
@scgopi
scgopi merged commit 98a603f into main Sep 3, 2026
1 check passed
@scgopi
scgopi deleted the remove-share-flag branch September 3, 2026 09:11
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