Skip to content

refactor effect runtime#18336

Open
tim-smart wants to merge 2 commits intoanomalyco:devfrom
tim-smart:effect-runtime
Open

refactor effect runtime#18336
tim-smart wants to merge 2 commits intoanomalyco:devfrom
tim-smart:effect-runtime

Conversation

@tim-smart
Copy link
Contributor

No description provided.

@github-actions github-actions bot added contributor needs:compliance This means the issue will auto-close after 2 hours. needs:title labels Mar 20, 2026
@github-actions
Copy link
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions
Copy link
Contributor

Hey! Your PR title refactor effect runtime doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@tim-smart
Copy link
Contributor Author

@kitlangton fyi was playing with the legos

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

Based on my search, I found several related PRs that are part of the same refactoring initiative:

Related PRs (not duplicates, but part of the same effort):

  1. PR effectify LSP service #18321 - "effectify LSP service" - effectify LSP service #18321
  2. PR effectify Pty service #18319 - "effectify Pty service" - effectify Pty service #18319
  3. PR effectify SessionStatus service #18313 - "effectify SessionStatus service" - effectify SessionStatus service #18313
  4. PR effectify Installation service, drop Effect suffix from namespaces #18266 - "effectify Installation service, drop Effect suffix from namespaces" - effectify Installation service, drop Effect suffix from namespaces #18266
  5. PR feat(bus): migrate Bus to Effect service with PubSub #18173 - "feat(bus): migrate Bus to Effect service with PubSub" - feat(bus): migrate Bus to Effect service with PubSub #18173

These PRs are all focused on migrating services to use the Effect library, which appears to be a coordinated refactoring effort related to the effect runtime changes in PR #18336. However, none of these are duplicates - they each address specific services being effectified.

@github-actions
Copy link
Contributor

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.

@github-actions github-actions bot removed the needs:compliance This means the issue will auto-close after 2 hours. label Mar 20, 2026
@github-actions github-actions bot closed this Mar 20, 2026
@kitlangton
Copy link
Contributor

Vouch @tim-smart

@kitlangton kitlangton reopened this Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants