fix: embed web UI in binary + rebrand wordmark to PentestCode#5
Merged
Conversation
Five CSS @import statements still referenced the pre-fork @opencode-ai/* package scope. The web app build (packages/app) failed to resolve them, so release binaries were compiled with --skip-embed-web-ui and the `web` command fell back to proxying http://localhost:3000 → HTTP 500. Rename the stale imports to @pentestcode/* in index.css and settings-v2.css so `bun run script/build.ts` embeds the web UI and `pentestcode web` serves it directly. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The web UI still showed "opencode": the Logo and WordmarkV2 pixel wordmarks, the <title>, and the PWA manifest name were all leftover from upstream, while the terminal banner already read "pentestcode". Redraw both SVG wordmarks to spell "pentestcode" — the six glyphs shared with "opencode" (p, e, n, c, o, d) are reused verbatim and translated into place, and t/s are drawn as rects in the same pixel-grid style. Logo keeps the two-tone emphasis (pentest = --icon-base, code = --icon-strong-base) and WordmarkV2's mask/filter widths grow to the new 11-glyph viewBox. Also set the document title and webmanifest name to PentestCode. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Owner
|
Hi, thanks for contribution! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The distributed binary returned HTTP 500 when starting the local web server (
pentestcode web).Root cause: the web UI was never embedded in the binary.
server/shared/ui.tsserves the UI from the build-embeddedopencode-web-ui.gen.ts, or — if absent — proxies tohttp://localhost:3000(dev server). The app build (packages/app) was broken because 5 CSS@imports still referenced the pre-fork@opencode-ai/*package scope (renamed to@pentestcode/*), so releases were compiled with--skip-embed-web-uiand thewebcommand fell back to the dead proxy → 500.Separately, the web UI still displayed "opencode" branding (wordmark, tab title, PWA name) while the terminal banner already read "pentestcode".
Changes
1. Restore web UI embedding (
5b0ef12)@opencode-ai/*→@pentestcode/*imports inpackages/app/src/index.cssandsettings-v2.css.bun run script/build.tsnow embeds the web UI;pentestcode webserves it directly.2. Rebrand web wordmark + title (
aac80b1)LogoandWordmarkV2SVGs to spell pentestcode. The six glyphs shared with "opencode" (p, e, n, c, o, d) are reused verbatim and translated into place;t/sare drawn as rects in the same pixel-grid style.Logokeeps the two-tone emphasis (pentest =--icon-base, code =--icon-strong-base);WordmarkV2mask/filter widths grow to the new 11-glyph viewBox.<title>and the PWA manifest name to PentestCode.Verification
Built a single native binary (no
--skip-embed-web-ui) and served it:GET /→ 200 with the embedded HTML (previously 500)/assets/index-*.js→ 200/global/health→ 200PentestCode; wordmark renders "pentestcode" in the browser.🤖 Generated with Claude Code