Skip to content

fix style.css path#492

Merged
penso merged 1 commit intomoltis-org:mainfrom
cyberpsyche:fix/assets_css_style
Mar 26, 2026
Merged

fix style.css path#492
penso merged 1 commit intomoltis-org:mainfrom
cyberpsyche:fix/assets_css_style

Conversation

@cyberpsyche
Copy link
Copy Markdown
Contributor

fix style.css path in crates/web/src/assets.rs

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 26, 2026

Greptile Summary

This PR fixes a single incorrect path in the compile-time CSS existence check in crates/web/src/assets.rs. The include_str! guard was pointing to assets/style.css when the generated stylesheet actually lives at assets/css/style.css. Without this fix, builds with the embedded-assets feature enabled would fail to compile after just build-css is run.\n\n- Corrects include_str!(\"assets/style.css\")include_str!(\"assets/css/style.css\") to match the real on-disk path crates/web/src/assets/css/style.css.\n- No other logic, behaviour, or serving paths are affected — the ASSETS embedded dir already recursively includes all of src/assets, so css/style.css was always correctly embedded for runtime use.

Confidence Score: 5/5

Safe to merge — one-line path fix with no logic changes or side effects.

The change is a minimal, targeted correction to a compile-time path string. The new path assets/css/style.css is confirmed to exist on disk at crates/web/src/assets/css/style.css. The include_str! macro is only used as a build-time guard and has no runtime impact. No other code paths are affected.

No files require special attention.

Important Files Changed

Filename Overview
crates/web/src/assets.rs Corrects the compile-time include_str! guard path from assets/style.css to assets/css/style.css, matching the actual file location on disk.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["cargo build --features embedded-assets"] --> B["include_dir! embeds src/assets/**"]
    A --> C["include_str! compile-time guard"]
    C -->|"before fix"| D["src/assets/style.css ❌ (not found)"]
    C -->|"after fix"| E["src/assets/css/style.css ✅"]
    E --> F["Compilation succeeds"]
    D --> G["Compilation error: file not found"]
Loading

Reviews (1): Last reviewed commit: "fix style.css path" | Re-trigger Greptile

@penso penso merged commit d437fcf into moltis-org:main Mar 26, 2026
1 check passed
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.

2 participants