Skip to content

fix: preserve POSIX path case in duplicate installation warnings - #707

Merged
sirmalloc merged 1 commit into
sirmalloc:mainfrom
vishnujayvel:oss/occ-6gr-ccstatusline-posix-path-case
Oct 29, 2026
Merged

sirmalloc merged 1 commit into
sirmalloc:mainfrom
vishnujayvel:oss/occ-6gr-ccstatusline-posix-path-case

Conversation

@vishnujayvel

Copy link
Copy Markdown
Contributor

fix: preserve POSIX path case in duplicate installation warnings

ccstatusline now warns when PATH contains installations in distinct POSIX directories such as /home/alice/Bin and /home/alice/bin. Directory comparisons retain case for POSIX paths and keep the existing case-insensitive handling of Windows drive paths and WSL mounts.

Type: bug fix · Risk: low — changes duplicate-installation detection only.

File Change
src/utils/global-package-manager.ts Preserves POSIX directory case when detecting duplicate installations.
src/utils/__tests__/global-package-manager.test.ts Covers distinct POSIX directories and case-insensitive Windows/WSL comparisons.

Validation:

  • Before the fix, the POSIX regression expected a duplicate-installation warning and received null.
  • bun test src/utils/__tests__/global-package-manager.test.ts src/utils/__tests__/global-command-resolution.test.ts: 18 passed, 0 failed.
  • bun run lint: passed TypeScript checking and ESLint.

@sirmalloc
sirmalloc merged commit 99d9f4d into sirmalloc:main Oct 29, 2026
3 checks passed
pcvelz added a commit to pcvelz/ccstatusline-usage that referenced this pull request Oct 29, 2026
Upstream: Extra Usage Limit widget (sirmalloc#756), live widget preview while picking (sirmalloc#755), Fast Mode widget (sirmalloc#754), transcript block fallback fixes (sirmalloc#752), editor keys on a global install (sirmalloc#751), git cache TTL (sirmalloc#750), npx cached copy skip (sirmalloc#749), Block Usage hide at 100% (sirmalloc#747), Block Reset Timer hide until 100% (sirmalloc#697), fish-style branch abbreviation (sirmalloc#614), jj detached HEAD (sirmalloc#705), hooks save warning (sirmalloc#703), dev reload (sirmalloc#700), merge ends at a hidden widget (sirmalloc#698), Block Forecast and Block Limit Timer widgets (sirmalloc#699), contributor guide (sirmalloc#706), POSIX path case fix (sirmalloc#707), whitespace-distinct conflict filenames (sirmalloc#709), Quick Start reorder (sirmalloc#715), bunx launch guidance (sirmalloc#714), jj diff summary counts (sirmalloc#711), powerline manual separators (sirmalloc#713), keychain credential by account (sirmalloc#532), Keychain probe timeout (sirmalloc#760), usage cache scoped per CLAUDE_CONFIG_DIR (sirmalloc#266), test isolation (sirmalloc#717, sirmalloc#753), dev-dependency bumps (sirmalloc#745)

Conflicts resolved per the standard-resolution table; fork deletions re-applied.

Hand edits outside conflicts:
- src/utils/global-command-resolution.ts: fork-rename (ccstatusline@ -> ccstatusline-usage@ in upstream's new bunx pattern)
- src/utils/__tests__/global-package-manager.test.ts: fork-rename (package dir ccstatusline-usage in upstream's new tests)
- src/utils/usage-fetch.ts: compat fix - cacheUsageData gained a fetchedAtMs arg (upstream sirmalloc#266); adapted the fork-only fetchFreshUsageData call site
- src/utils/__tests__/usage-fetch.test.ts: compat fixes - cache/lock now hashed per CLAUDE_CONFIG_DIR (usageFilePath helper) in four fork tests; upstream fableUsage renamed to fork's weeklyFableUsage; seedUsageCache takes TokenHome
- src/utils/__tests__/usage-prefetch.test.ts: compat fix - mockFetchUsageData renamed to upstream's mockFetchUsage in cleanly-merged tests
- README.md: took upstream's Quick Start move (sirmalloc#715) and dropped the fork's old copy; fork-rename of its ccstatusline@ specifiers; links to fork-deleted docs (docs/USAGE.md, docs/WINDOWS.md) replaced with the fork's inline text
- CONTRIBUTING.md: links to the fork-deleted docs/DEVELOPMENT.md pointed at the README Development section
- docs/test-retirements.md: ledger entries for the 12 upstream test renames (sirmalloc#697, sirmalloc#698, sirmalloc#700, sirmalloc#705, sirmalloc#713, sirmalloc#760)
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