opencode icon indicating copy to clipboard operation
opencode copied to clipboard

fix(app): repair web manifest and OAuth errors; fix Windows typings

Open Observer-GGboy opened this issue 5 days ago • 5 comments

What does this PR do?

  • Fix invalid site.webmanifest that caused a manifest parse error in the web UI.
  • Improve OAuth error visibility in the connect dialog and include response details for token exchange failures.
  • Replace Windows placeholder custom-elements.d.ts files with actual typings in app/enterprise packages.

How did you verify your code works?

  • bun turbo typecheck Fixes #8812

Observer-GGboy avatar Jan 16 '26 05:01 Observer-GGboy

Hey! Your PR title Fix web manifest, OAuth error visibility, and Windows custom element typings 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.

github-actions[bot] avatar Jan 16 '26 05:01 github-actions[bot]

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

No duplicate PRs found.

github-actions[bot] avatar Jan 16 '26 05:01 github-actions[bot]

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

github-actions[bot] avatar Jan 16 '26 05:01 github-actions[bot]

@Observer-GGboy why the change to the codex plugin?

adamdotdevin avatar Jan 16 '26 16:01 adamdotdevin

@Observer-GGboy if you can revert the changes to codex.ts then i'll merge

adamdotdevin avatar Jan 18 '26 11:01 adamdotdevin