Skip to content

fix(web): keep the final segment of directory paths with a trailing separator - #5460

Merged
juliusmarminge merged 1 commit into
pingdotgg:mainfrom
jorvarea:fix/directory-chip-trailing-separator
Aug 15, 2026
Merged

juliusmarminge merged 1 commit into
pingdotgg:mainfrom
jorvarea:fix/directory-chip-trailing-separator

fix(web): keep the final segment of directory paths with a trailing s…

cdbd70f
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - UI Consistency succeeded Aug 15, 2026 in 23s

UI Consistency: No findings

All clear

Details

Note

Your check run agent prompt is: .macroscope/check-run-agents/ui-consistency.md
More information about how Check Run Agents work can be found in our Help Center.


🔍 Agent activity — reasoning, no tool calls

💭 I'm examining the logic fix in the markdown-links.ts file, specifically the basenameOfPath function, and tracing through an edge case where the path is "/" to verify the fix handles it correctly—though the test seems to reference "/tmp/" instead of the root path. The fix doesn't introduce a regression for the root case since the original code would also produce an empty label. I'm also checking Windows drive roots like "C:" which should return "C:" correctly. Since this is purely a TypeScript logic fix with tests and no UI or CSS changes, it falls outside the scope of the consistency check, so I should report this as all clear. This is a straightforward logic fix with no UI implications, so it's within scope. I'm noting that other files like pierre-icons.ts might have the same issue, but addressing that would be unrelated cleanup outside the current task.


Agent Credits: 20 credits