fix(theme): AI agent icon 404 - point at ai-agent-icon.svg #21

Merged

mohlec merged 1 commit from fix/ai-agent-icon-404 into slash/main

2026-08-04 20:09:05 +00:00

mohlec commented

2026-08-04 17:50:33 +00:00

Owner

Fixes #20.

slash-theme.css overrode the Agents app-bar icon with url("/static/slash-site-logo.png"), but that file is never emitted to the client root. webpack.config.js CopyPlugin copies slash-theme.css and ai-agent-icon.svg to the root, but there is no pattern for slash-site-logo.png - it only exists under src/images/ where it is consumed as a webpack import by product_branding_free_edition.tsx. A bare url() in a copied-verbatim CSS file gets no such resolution, so it resolves to a path that does not exist.

Verified on the live deploy:

/static/ai-agent-icon.svg  -> 200
/static/slash-theme.css    -> 200
/static/slash-site-logo.png -> 404

So the correct asset is already shipping and already reachable on the current build - this is a one-line repoint, with no new asset and no CopyPlugin change needed.

Also picks up the intent of the two theme-repo issues that were circling this: use the SVG rather than the PNG, and wire ai-agent-icon.svg into the AI plugin icon.

Verify after deploy: the Agents icon in the app bar renders the Slash mark instead of a blank/default box.

Fixes #20. `slash-theme.css` overrode the Agents app-bar icon with `url("/static/slash-site-logo.png")`, but that file is **never emitted to the client root**. `webpack.config.js` CopyPlugin copies `slash-theme.css` and `ai-agent-icon.svg` to the root, but there is no pattern for `slash-site-logo.png` - it only exists under `src/images/` where it is consumed as a webpack import by `product_branding_free_edition.tsx`. A bare `url()` in a copied-verbatim CSS file gets no such resolution, so it resolves to a path that does not exist. Verified on the live deploy: ``` /static/ai-agent-icon.svg -> 200 /static/slash-theme.css -> 200 /static/slash-site-logo.png -> 404 ``` So the correct asset is **already shipping and already reachable on the current build** - this is a one-line repoint, with no new asset and no CopyPlugin change needed. Also picks up the intent of the two theme-repo issues that were circling this: use the SVG rather than the PNG, and wire `ai-agent-icon.svg` into the AI plugin icon. Verify after deploy: the Agents icon in the app bar renders the Slash mark instead of a blank/default box.

mohlec added 1 commit

2026-08-04 17:50:33 +00:00

fix(theme): point AI agent icon at ai-agent-icon.svg, not the unshipped PNG

Build Slash Webapp / branding-audit (pull_request) Successful in 44s

Details

Build Slash Webapp / build-webapp (pull_request) Has been skipped

Details

64005c477d

slash-theme.css referenced /static/slash-site-logo.png, which is never
emitted to the client root - webpack's CopyPlugin copies slash-theme.css
and ai-agent-icon.svg there, but not slash-site-logo.png. The URL 404s
in production and beta users see a broken Agents app-bar icon.

ai-agent-icon.svg is already copied to the client root and already
serves 200 on the current deployed build, so this needs no new asset
and no CopyPlugin change.

Closes #20

mohlec merged commit 6c14d63742 into slash/main

2026-08-04 20:09:05 +00:00

mohlec deleted branch fix/ai-agent-icon-404

2026-08-04 20:09:06 +00:00

mohlec referenced this pull request from a commit

2026-08-04 20:09:07 +00:00

Merge pull request 'fix(theme): AI agent icon 404 - point at ai-agent-icon.svg' (#21) from fix/ai-agent-icon-404 into slash/main

mohlec referenced this pull request

2026-08-05 01:09:25 +00:00

CI: build-server/build-webapp do not run on pull requests, so compile errors only surface after merge #23

mohlec referenced this pull request

2026-08-05 01:28:11 +00:00