Upgrade Effect 4 cohort to beta.99 - #1447
Merged
schickling-assistant merged 2 commits intoJul 19, 2026
Merged
Conversation
Contributor
PR preview
Report historyPR 1447 · 2026-07-19 18:57 UTC
PR 1447 · 2026-07-19 14:57 UTC
PR 1447 · 2026-07-19 12:13 UTC
PR 1447 · 2026-07-19 12:03 UTC
PR 1447 · 2026-07-19 11:52 UTC
PR 1447 · 2026-07-19 10:57 UTC
PR 1447 · 2026-07-19 10:42 UTC
PR 1447 · 2026-07-19 10:34 UTC
|
Collaborator
Author
|
Temporarily closing and reopening the unchanged head to trigger the current base Posted on behalf of @schickling
|
Collaborator
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5c896cff9e
ℹ️ 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".
schickling-assistant
enabled auto-merge (squash)
July 19, 2026 15:00
schickling-assistant
disabled auto-merge
July 19, 2026 15:07
schickling-assistant
force-pushed
the
schickling-assistant/chore/effect-beta99
branch
from
July 19, 2026 18:32
1d789fb to
33b9cae
Compare
schickling-assistant
enabled auto-merge (squash)
July 19, 2026 18:35
schickling-assistant
deleted the
schickling-assistant/chore/effect-beta99
branch
July 19, 2026 18:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Problem
LiveStore's Effect 4 cohort is pinned to
4.0.0-beta.97. Effect beta.99 removes privateGraphfields used by two internal helpers and adds a duplicate-package diagnostic that detects the intentionally mixed published/workspace dependency graph in the docs fixture project.Scope
This PR is strictly the beta.97 to beta.99 migration:
Graphfield access with public APIsHistoryDag's degree-local traversal and insertion order with internal adjacency indexes@effect/platformsplit only in the docs fixture's direct and solution TypeScript configsThere are no lifecycle, shutdown, tracing, Metro, DevTools artifact, release workflow, or Nix source-authority changes in this PR.
Behavior
No behavior change is intended. The only runtime source changes are compatibility adaptations for removed private
Graphfields:HistoryDagretains parallel-edge order, inbound/outbound lookup, incident-edge removal, self-loop handling, and copy isolation.Debugsnapshots publicGraph.nodesindices before removing filtered nodes, matching the previous mutable-map iteration behavior.Split ownership
Verification
Exact head:
33b9cae6ffd6d1ca0fdee0e33cc85cf98b659aa2devenv tasks run check:quick --mode before --no-tui --show-output: passeddevenv tasks run test:unit --mode before --no-tui --show-output: 659 passed, 17 skipped, 1 expected failure@livestore/utilsand@livestore/common: passedThe GitHub CI matrix on the exact head is the final integration and browser E2E gate.
References
Closes #1446.
Posted on behalf of @schickling
agent_nameagent_session_idagent_toolagent_tool_versionagent_runtimeagent_modelruntime_profileskills_manifestworktreemachinetooling_profile