Skip to content

fix(opencode): add ulid dependency (#1988) - #1989

Merged
thdxr merged 2 commits into
anomalyco:devfrom
delafthi:push-puprqsznxozv
Aug 17, 2025
Merged

thdxr merged 2 commits into
anomalyco:devfrom
delafthi:push-puprqsznxozv

Conversation

@delafthi

Copy link
Copy Markdown
Contributor

See #1988

@rekram1-node

rekram1-node commented Aug 16, 2025 •

Copy link
Copy Markdown
Collaborator

if ulid is shared between multiple packages should it be installed in root package.json in catalog, and then referenced with catalog reference there?

@delafthi

delafthi commented Aug 16, 2025 •

Copy link
Copy Markdown
Contributor Author

I ran into this issue in #1392 with @octokit/rest — it was fixed in #1396 using a similar change. That's why I simply added ulid to the opencode package, but I'm not experienced at all with bun. So, if you think catalog is a better approach I'll make the change.

@rekram1-node

Copy link
Copy Markdown
Collaborator

Yeah I think catalog is correct approach here, I mean I could be wrong but that seems standard elsewhere

@thdxr
thdxr merged commit 5d5ac16 into anomalyco:dev Aug 17, 2025
@delafthi
delafthi deleted the push-puprqsznxozv branch August 17, 2025 07:12
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request May 5, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
meSingh pushed a commit to meSingh/opencode that referenced this pull request Sep 6, 2026
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.

3 participants