Skip to content

chore(installer): Update installer for async registration flow - #708

Merged
Steve-Mcl merged 12 commits into
mainfrom
693-device-installer-updates
Jul 30, 2026
Merged

chore(installer): Update installer for async registration flow#708
Steve-Mcl merged 12 commits into
mainfrom
693-device-installer-updates

Conversation

@knolleary

@knolleary knolleary commented Jul 29, 2026

Copy link
Copy Markdown
Member

Update the installer to work with the new async registration flow.

There is a big chicken/egg issue when it comes to testing this PR. It installs latest device agent code from npm - that's no good when you want to test installer changes alongside unpublished device agent changes.

I found a hacky workaround by temporarily modifying the installer code to install from a hardcoded local path rather than npm. Not entirely satisfactory.

But, it is true to say these installer updates do not need to merge at the same time as the setup PR. We can do a release of the updated device agent, then finish verification of the installer against the newly published version.

The main changes are:

  1. Adds some utils for formatting console output (similar to chalk in the node.js code). This allows for more consistent output. But - see the note below about future work.
  2. The installer no longer requires the OTC being provided - as the new device agent can handle that for you.
  3. It disables the ability to paste in a full device.yml file - again, see below.

### Future work

The original intent was to rework the whole sequence of steps the installer takes - to give the user more control. I have a local branch with that 90% done, but its a lot of work to fully test. So I've reduced the scope a lot to keep the same level of opinionated behaviour, but with the async registration supported.

I will raise separate issues to identify what needs doing next, but that doesn't block this iteration.

@knolleary knolleary changed the title 693 device installer updates Update installer for new async registration flow Jul 29, 2026
@knolleary
knolleary requested a review from Steve-Mcl July 29, 2026 09:35
@knolleary knolleary changed the title Update installer for new async registration flow feat(installer): Update installer for new async registration flow Jul 29, 2026
Base automatically changed from 693-async-registration to main July 29, 2026 13:31
@knolleary

Copy link
Copy Markdown
Member Author

Given the installer release doesn't use release-please currently, I think I should make this a chore: PR to avoid bumping the Device Agent version unnecessarily.

@Steve-Mcl

Copy link
Copy Markdown
Contributor

Given the installer release doesn't use release-please currently, I think I should make this a chore: PR to avoid bumping the Device Agent version unnecessarily.

Agree

@Steve-Mcl Steve-Mcl changed the title feat(installer): Update installer for new async registration flow chore(installer): Update installer for async registration flow Jul 30, 2026
@Steve-Mcl
Steve-Mcl merged commit 82de907 into main Jul 30, 2026
17 checks passed
@Steve-Mcl
Steve-Mcl deleted the 693-device-installer-updates branch July 30, 2026 09:01
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