Fix generated pack index URL to reference output directory

[edriouk]

Fixes

• Fixes [Bug]: cpackget add crashes when updating a filesystem hosted .Web/index.pidx generated by vidx2pidx #835

Changes

• Generate the pack index URL from the output file’s directory instead of including the output filename.
• Normalize the file URL with a trailing slash while preserving the root file:/// URL.
• Update CLI and PidxXML.Update tests to verify the corrected URL format across platforms.

Risk / Limitations

• Low risk; the change is limited to generated pack index URL metadata. Consumers relying on the previous filename-inclusive URL will receive the corrected directory URL.

Checklist

• 🤖 This change is covered by unit tests (if applicable).
• 🤹 Manual testing has been performed (not necessary; covered by unit tests).
• 🛡️ Security impacts have been considered (no impact).
• 📖 Documentation updates are complete (not required).
• 🧠 Third-party dependencies and TPIP updated (not required).

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[qltysh]

Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (1)

Rating	File	% Diff	Uncovered Line #s
	cmd/pidx.go	100.0%
	Total	100.0%

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[github-actions]

Test Results

 2 files  ±0   2 suites  ±0   0s ⏱️ ±0s
45 tests ±0  45 ✅ ±0  0 💤 ±0  0 ❌ ±0 
90 runs  ±0  90 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9309687. ± Comparison against base commit 86f3c25.

[Copilot]

🟢 Approval recommended

The URL fix and corresponding tests are complete with no unresolved issues.

Pull request overview

Fixes generated pack index URLs to reference the output directory with normalized file URL formatting.

Changes:

• Generate URLs from the output directory.
• Preserve file:/// for filesystem roots.
• Update platform-specific CLI and update tests.

File summaries

File	Description
cmd/pidx.go	Corrects generated directory-based file URLs.
cmd/pidx_test.go	Verifies directory URLs across platforms.
cmd/cli_test.go	Updates CLI output expectations.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[brondani]

LGTM