Skip to content

enhancement(cli): #1690 eliminate api route instantiation during greenwood lifecycles#1691

Merged
thescientist13 merged 1 commit into
masterfrom
enhancement/issue-1690-no-side-effectul-api-routes
Jun 18, 2026
Merged

enhancement(cli): #1690 eliminate api route instantiation during greenwood lifecycles#1691
thescientist13 merged 1 commit into
masterfrom
enhancement/issue-1690-no-side-effectul-api-routes

Conversation

@thescientist13

@thescientist13 thescientist13 commented Jun 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Related Issue

resolves #1690

Documentation

N / A

Summary of Changes

  1. Don't import API routes during graph lifecycle

TODO

  1. Track a better way to do this - put this into my personal backlog for now
    • for example, would this line with a comment still lead to a false positive? - // export const isolation = true

@thescientist13 thescientist13 added enhancement Improve something existing (e.g. no docs, new APIs, etc) CLI labels Jun 13, 2026
@thescientist13 thescientist13 changed the title enhancement(cli): #1690 eliminate api route instantiation during gree… enhancement(cli): #1690 eliminate api route instantiation during greenwood lifecycles Jun 13, 2026
@thescientist13 thescientist13 mentioned this pull request Jun 17, 2026
Open
9 tasks
@thescientist13 thescientist13 merged commit 3bf30d3 into master Jun 18, 2026
10 checks passed
@thescientist13 thescientist13 deleted the enhancement/issue-1690-no-side-effectul-api-routes branch June 18, 2026 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

CLI enhancement Improve something existing (e.g. no docs, new APIs, etc)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

side effects in API routes should not break the build

1 participant

@thescientist13