Skip to content

Conversation

@shuv1337
Copy link
Collaborator

Summary

  • Sync shuvcode fork with upstream sst/opencode v1.0.177
  • Resolves merge conflicts from upstream changes

Resolved Conflicts

  • package.json files: Accepted upstream version 1.0.177
  • bun.lock: Regenerated with new dependencies
  • .github/workflows/publish.yml: Kept fork-specific workflow
  • packages/extensions/zed/extension.toml: Accepted upstream
  • packages/desktop/src/context/global-sdk.tsx: Merged fork's fetch/throwOnError with upstream's signal change
  • packages/desktop/src/context/global-sync.tsx: Merged upstream retry() wrappers with fork's defensive array check
  • packages/desktop/src/pages/layout.tsx: Kept fork's inline header (upstream refactored to separate component)

Preserved Fork Features

  • Desktop inline header with shuvcode ASCII branding (AsciiLogo/AsciiMark)
  • ThemePicker and FontPicker components in header
  • Mobile session dropdown with viewport constraints
  • Version display in sidebar
  • Defensive array handling in global-sync for API responses
  • Custom fetch/throwOnError configuration in global-sdk

Validation

  • ✅ All type checks pass
  • ✅ All tests pass (329 passed, 1 skipped)

Closes #163

Frank and others added 22 commits December 20, 2025 01:26
Resolved conflicts:
- package.json files: accepted upstream version 1.0.177
- bun.lock: regenerated
- .github/workflows/publish.yml: kept fork-specific workflow
- packages/extensions/zed/extension.toml: accepted upstream
- packages/desktop/src/context/global-sdk.tsx: merged fork's fetch/throwOnError with upstream's signal change
- packages/desktop/src/context/global-sync.tsx: merged upstream retry() wrappers with fork's defensive array check
- packages/desktop/src/pages/layout.tsx: kept fork's inline header with theme/font pickers (upstream refactored to separate component)

Preserved fork features:
- Desktop inline header with shuvcode ASCII branding
- ThemePicker and FontPicker components
- Mobile session dropdown
- Version display
- Defensive array handling in global-sync
- Custom fetch/throwOnError in global-sdk
@coderabbitai
Copy link

coderabbitai bot commented Dec 20, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch opencode/issue163-20251220150233

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shuv1337 shuv1337 merged commit 1a138d2 into integration Dec 20, 2025
1 of 3 checks passed
@shuv1337 shuv1337 deleted the opencode/issue163-20251220150233 branch December 20, 2025 15:07
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.

[Upstream Sync] Merge conflict with v1.0.177

5 participants