Skip to content

Improve error messages when the local package store is not found#43

Merged
astolcenburg merged 1 commit into
rdkcentral:mainfrom
astolcenburg:main
Jun 12, 2026
Merged

Improve error messages when the local package store is not found#43
astolcenburg merged 1 commit into
rdkcentral:mainfrom
astolcenburg:main

Conversation

@astolcenburg

Copy link
Copy Markdown
Contributor

Ref: #RDKEAPPRT-839

Copilot AI review requested due to automatic review settings June 12, 2026 12:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves CLI error reporting when a local package store cannot be located, by capturing and displaying the directory candidates that were searched. This makes failures in bolt push, bolt fetch, and bolt make --install more actionable for users.

Changes:

  • Extend PackageStore.find() to optionally record each candidate bolts/ directory it checks while walking up the filesystem.
  • Improve bolt fetch and bolt make --install errors to include the searched paths when the store is missing.
  • Improve bolt push errors to additionally report store search paths when the package name lookup requires a store but none is found.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
bolt/src/push.cjs Threads store-search tracing into package lookup and conditionally appends store search info to the “package not found” error.
bolt/src/PackageStore.cjs Adds optional searched output parameter to record searched store directories during discovery.
bolt/src/make.cjs Enhances install-mode failure message to include all searched store locations.
bolt/src/fetch.cjs Enhances missing local store failure message to include all searched store locations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@astolcenburg astolcenburg requested a review from ansu-mathew June 12, 2026 12:06
@astolcenburg astolcenburg merged commit 78d074b into rdkcentral:main Jun 12, 2026
3 of 4 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants