Skip to content

Fix installer validation and PATH diagnostics - #2682

Merged
Widthdom merged 4 commits into
mainfrom
codex/fix-issue1751-2014-2011-1766
May 25, 2026
Merged

Fix installer validation and PATH diagnostics#2682
Widthdom merged 4 commits into
mainfrom
codex/fix-issue1751-2014-2011-1766

Conversation

@Widthdom

@Widthdom Widthdom commented May 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Verify staged and installed cdidx binaries with --version before reporting install success.
  • Add TMPDIR writability/free-space preflight and safe atomic temp probe handling.
  • Detect PATH shadowing and add opt-in shell profile PATH updates via CDIDX_INSTALL_UPDATE_PATH=1.

Documentation / Changelog

  • Added changelog.d/unreleased/1751.fixed.md
  • Added changelog.d/unreleased/1766.fixed.md
  • Added changelog.d/unreleased/2011.fixed.md
  • Added changelog.d/unreleased/2014.fixed.md

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~InstallScriptTests -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • bash -n install.sh
  • Adversarial review: No blocking/actionable issues found.

Follow-up candidates

  • None.

Fixes #1751
Fixes #2014
Fixes #2011
Fixes #1766

@Widthdom
Widthdom marked this pull request as ready for review May 25, 2026 03:25
@Widthdom
Widthdom merged commit b6a622c into main May 25, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1751-2014-2011-1766 branch May 25, 2026 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment