Extend refresh with new options

[dmitry-lyfar]

Description

• workshop refresh will skip the change if no updates to the SDKs, interface connections or the base were found by the resolver.

• workshop refresh --restore will restore the workshop from the latest snapshot made after the last successful setup-base run.

Self-review quick check

• Make decisions that cost a lot to reverse explicit in the PR description.
• Avoid nested conditions.
• Delete dead code and redundant comments.
• Normalise symmetries by sticking to doing identical things identically.

// one way to handle errors
if err := f(); err != nil {
   ...
}

// one way to handle multiple returns
val, err := f()
if err != nil {
   ...
}
...

• Check that coupled code elements, files, and directories are adjacent. For example, test data is stored as close as possible to a test.
• Put variable declaration and initialisation together.
• Divide large expressions into digestable and self-explanatory ones. Use multiple variables if required.
• Put a blank line between two logically different chunks of code.
• Follow the style guide for new error messages.

Docs

• I have checked and added or updated relevant documentation.
• I have checked and added or updated relevant release notes.
• I have included the technical author in the review.

Or:

• I confirm the PR has no implications for documentation.

[Copilot]

Pull Request Overview

This PR introduces a refresh-restore feature that allows workshops to be refreshed in two modes: update mode (default) that applies changes from workshop definitions, and restore mode that reverts workshops to their last successful snapshot state. The feature includes new CLI options, API parameters, and logic to detect when no updates are available.

Key changes:

• Adds --restore flag to refresh command for reverting to snapshot state
• Implements refresh option validation in API with "update" and "restore" modes
• Introduces "no-updates-available" error handling to prevent unnecessary operations

Reviewed Changes

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

Show a summary per file

File	Description
internal/overlord/conflict/conflict.go	Defines RefreshOption enum with update/restore modes and parsing logic
internal/overlord/workshopstate/request.go	Implements refresh option handling and update detection logic in RefreshMany method
internal/daemon/api_workshops.go	Adds refresh option validation and no-updates-available response handling
cmd/workshop/refresh.go	Adds --restore flag and integrates refresh options into CLI
cmd/workshop/sketch.go	Handles no-updates-available errors in sketch refresh workflow
client/projects.go	Updates Refresh API call to include refresh option parameter
Multiple test files	Updates test expectations to include new refresh-option parameter

[jonathan-conder]

Looks good and seems to be working well for the most part

[github-actions]

TICS Quality Gate

✔️ Passed

workshop

All conditions passed

See the results in the TICS Viewer

The following files have been checked for this project

• client/client.go
• client/projects.go
• cmd/workshop/refresh.go
• cmd/workshop/sketch.go
• internal/daemon/api_workshops.go
• internal/daemon/response.go
• internal/overlord/conflict/conflict.go
• internal/overlord/sdkstate/handlers.go
• internal/overlord/workshopstate/request.go

Automatic-tests / code-coverage / TICS GitHub Action