Skip to content

Add fetch command#27

Merged
astolcenburg merged 1 commit into
rdkcentral:mainfrom
astolcenburg:feature/RDKEAPPRT-689
Apr 7, 2026
Merged

Add fetch command#27
astolcenburg merged 1 commit into
rdkcentral:mainfrom
astolcenburg:feature/RDKEAPPRT-689

Conversation

@astolcenburg
Copy link
Copy Markdown
Contributor

Ref: #RDKEAPPRT-689

Copilot AI review requested due to automatic review settings April 7, 2026 11:23
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new bolt fetch CLI command to download bolt packages from a remote “package store” into the local package store, with support for multiple fetch backends (built-in and user plugins) and accompanying documentation.

Changes:

  • Introduces bolt fetch command wiring, CLI option parsing, and global config keys for package store settings.
  • Implements fetch backends: basic (direct file download) and rdk (session/cookie-based login + download), plus a plugin mechanism.
  • Adds user documentation for bolt fetch and updates existing docs/README/help text.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
bolt/src/globalConfig.cjs Adds parsing for packageStoreURL, packageStoreType, packageStoreUser from ~/.bolt/config.json.
bolt/src/fetch.cjs New core implementation for fetching, downloading, auth prompting helpers, and plugin loading.
bolt/src/fetch-basic.cjs New “basic” fetch backend (no auth; static file server).
bolt/src/fetch-rdk.cjs New “rdk” backend (login + cookie reuse; re-auth on expiry).
bolt/src/bolt.cjs Registers fetch command and adds help text.
bolt/README.md Documents bolt fetch usage and adds packageStoreURL to global config docs.
bolt/docs/local-package-store.md Notes local package store is used by bolt fetch.
bolt/docs/fetch.md New detailed documentation for bolt fetch and custom fetch plugins.

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

Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/README.md
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.


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

Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.


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

Comment thread bolt/src/globalConfig.cjs Outdated
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/docs/fetch.md
Comment thread bolt/docs/fetch.md
Comment thread bolt/docs/fetch.md Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.


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

Comment thread bolt/src/globalConfig.cjs Outdated
Comment thread bolt/src/fetch.cjs
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.


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

Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/src/fetch.cjs Outdated
Comment thread bolt/docs/fetch.md
Comment thread bolt/src/fetch.cjs
Ref: #RDKEAPPRT-689
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.


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

Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs
Comment thread bolt/src/fetch.cjs
@astolcenburg astolcenburg requested a review from ansu-mathew April 7, 2026 14:47
@astolcenburg astolcenburg merged commit c65fda9 into rdkcentral:main Apr 7, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 7, 2026
@astolcenburg astolcenburg deleted the feature/RDKEAPPRT-689 branch April 7, 2026 15:14
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