feat: compile and upload sketch

* feat: compile and upload sketch

* imoplement code review suggestion

* use grpc instead of exec

* try update golangci-lint

* fixup! Fixed linter check

* fixup! Fixed linter check

* add suggested comment

* Update how to run examples in the README

* add error if no board are detected

* chore format

* Update internal/orchestrator/orchestrator.go

Co-authored-by: Davide <davideneri18@gmail.com>

---------

Co-authored-by: Xayton <30591904+Xayton@users.noreply.github.com>
Co-authored-by: Davide <davideneri18@gmail.com>

Author: 3 people

.env

@@ -29,3 +29,5 @@ go.work.sum
 build/
 
 **/.cache
+
+orchestrator

.gitignore

@@ -8,6 +8,7 @@ import (
 	"log/slog"
 	"net/http"
 	"os"
+	"path"
 	"time"
 
 	dockerClient "github.com/docker/docker/client"
@@ -19,19 +20,25 @@ import (
 	"github.com/arduino/arduino-app-cli/pkg/parser"
 )
 
+const DockerRegistry = "ghcr.io/bcmi-labs/"
+const DockerPythonImage = "arduino/appslab-python-apps-base:0.0.2"
+
 var pythonImage string
 
 func init() {
 	// Registry base: contains the registry and namespace, common to all Arduino docker images.
 	registryBase := os.Getenv("DOCKER_REGISTRY_BASE")
+	if registryBase == "" {
+		registryBase = DockerRegistry
+	}
 
 	// Python image: image name (repository) and optionally a tag.
 	pythonImageAndTag := os.Getenv("DOCKER_PYTHON_BASE_IMAGE")
-
-	pythonImage = registryBase
-	if pythonImageAndTag != "" {
-		pythonImage += pythonImageAndTag
+	if pythonImageAndTag == "" {
+		pythonImageAndTag = DockerPythonImage
 	}
+
+	pythonImage = path.Join(registryBase, pythonImageAndTag)
 	fmt.Println("Using pythonImage:", pythonImage)
 }
 
@@ -78,7 +85,10 @@ func main() {
 			Use:   "start",
 			Short: "Start the Python app",
 			Run: func(cmd *cobra.Command, args []string) {
-				provisionHandler(cmd.Context(), docker, parsedApp)
+				if parsedApp.MainPythonFile != nil {
+					provisionHandler(cmd.Context(), docker, parsedApp)
+				}
+
 				startHandler(cmd.Context(), parsedApp)
 			},
 		},

cmd/orchestrator/main.go

@@ -0,0 +1,3 @@
+display-name: Just blink
+description: Very simple blink example
+

examples/justblink/app.yaml

@@ -0,0 +1,13 @@
+#include <Arduino.h>
+
+void setup() {
+  pinMode(LED_BUILTIN, OUTPUT);
+}
+
+void loop() {
+  digitalWrite(LED_BUILTIN, HIGH);
+  delay(1000); // Wait for 1 second.
+
+  digitalWrite(LED_BUILTIN, LOW);
+  delay(1000); // Wait for 1 second.
+}

examples/justblink/sketch/sketch.ino

@@ -1,13 +1,15 @@
 module github.com/arduino/arduino-app-cli
 
-go 1.24
+go 1.24.1
 
 require (
-	github.com/arduino/go-paths-helper v1.12.1
-	github.com/davecgh/go-spew v1.1.1
-	github.com/djherbis/buffer v1.1.0
+	github.com/arduino/arduino-cli v1.2.2
+	github.com/arduino/go-paths-helper v1.13.0
+	github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc
+	github.com/djherbis/buffer v1.2.0
 	github.com/djherbis/nio/v3 v3.0.1
 	github.com/docker/docker v28.0.4+incompatible
+	github.com/sirupsen/logrus v1.9.3
 	github.com/spf13/cobra v1.9.1
 	github.com/stretchr/testify v1.10.0
 	github.com/vmihailenco/msgpack/v5 v5.4.1
@@ -18,36 +20,90 @@ require (
 )
 
 require (
-	github.com/creack/goselect v0.1.2 // indirect
-	github.com/vmihailenco/tagparser/v2 v2.0.0 // indirect
-)
-
-require (
+	dario.cat/mergo v1.0.0 // indirect
+	fortio.org/safecast v1.0.0 // indirect
 	github.com/Microsoft/go-winio v0.6.2 // indirect
+	github.com/ProtonMail/go-crypto v1.2.0 // indirect
+	github.com/arduino/go-properties-orderedmap v1.8.1 // indirect
+	github.com/arduino/go-serial-utils v0.1.2 // indirect
+	github.com/arduino/go-timeutils v0.0.0-20171220113728-d1dd9e313b1b // indirect
+	github.com/arduino/go-win32-utils v1.0.0 // indirect
+	github.com/arduino/pluggable-discovery-protocol-handler/v2 v2.2.1 // indirect
+	github.com/cloudflare/circl v1.6.1 // indirect
+	github.com/cmaglie/pb v1.0.27 // indirect
+	github.com/codeclysm/extract/v4 v4.0.0 // indirect
 	github.com/containerd/log v0.1.0 // indirect
+	github.com/creack/goselect v0.1.2 // indirect
+	github.com/cyphar/filepath-securejoin v0.4.1 // indirect
 	github.com/distribution/reference v0.6.0 // indirect
 	github.com/docker/go-connections v0.5.0 // indirect
 	github.com/docker/go-units v0.5.0 // indirect
+	github.com/emirpasic/gods v1.18.1 // indirect
 	github.com/felixge/httpsnoop v1.0.4 // indirect
+	github.com/fsnotify/fsnotify v1.8.0 // indirect
+	github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 // indirect
+	github.com/go-git/go-billy/v5 v5.6.2 // indirect
+	github.com/go-git/go-git/v5 v5.16.0 // indirect
 	github.com/go-logr/logr v1.4.2 // indirect
 	github.com/go-logr/stdr v1.2.2 // indirect
+	github.com/go-viper/mapstructure/v2 v2.2.1 // indirect
+	github.com/gofrs/uuid/v5 v5.3.2 // indirect
 	github.com/gogo/protobuf v1.3.2 // indirect
+	github.com/golang/groupcache v0.0.0-20241129210726-2c02b8208cf8 // indirect
+	github.com/h2non/filetype v1.1.3 // indirect
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect
+	github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 // indirect
+	github.com/josharian/intern v1.0.0 // indirect
+	github.com/juju/errors v1.0.0 // indirect
+	github.com/kevinburke/ssh_config v1.2.0 // indirect
+	github.com/klauspost/compress v1.17.2 // indirect
+	github.com/leonelquinteros/gotext v1.7.1 // indirect
+	github.com/mailru/easyjson v0.7.7 // indirect
+	github.com/marcinbor85/gohex v0.0.0-20210308104911-55fb1c624d84 // indirect
+	github.com/mattn/go-isatty v0.0.20 // indirect
+	github.com/mattn/go-runewidth v0.0.15 // indirect
 	github.com/moby/docker-image-spec v1.3.1 // indirect
 	github.com/moby/term v0.5.2 // indirect
 	github.com/morikuni/aec v1.0.0 // indirect
 	github.com/opencontainers/go-digest v1.0.0 // indirect
 	github.com/opencontainers/image-spec v1.1.1 // indirect
+	github.com/pelletier/go-toml/v2 v2.2.3 // indirect
+	github.com/pjbgf/sha1cd v0.3.2 // indirect
 	github.com/pkg/errors v0.9.1 // indirect
-	github.com/pmezard/go-difflib v1.0.0 // indirect
+	github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
+	github.com/rivo/uniseg v0.4.4 // indirect
+	github.com/sagikazarmark/locafero v0.7.0 // indirect
+	github.com/schollz/closestmatch v2.1.0+incompatible // indirect
+	github.com/sergi/go-diff v1.3.2-0.20230802210424-5b0b94c5c0d3 // indirect
+	github.com/skeema/knownhosts v1.3.1 // indirect
+	github.com/sourcegraph/conc v0.3.0 // indirect
+	github.com/spf13/afero v1.12.0 // indirect
+	github.com/spf13/cast v1.7.1 // indirect
 	github.com/spf13/pflag v1.0.6 // indirect
+	github.com/spf13/viper v1.20.1 // indirect
+	github.com/subosito/gotenv v1.6.0 // indirect
+	github.com/ulikunitz/xz v0.5.12 // indirect
+	github.com/vmihailenco/tagparser/v2 v2.0.0 // indirect
+	github.com/xanzy/ssh-agent v0.3.3 // indirect
+	go.bug.st/downloader/v2 v2.2.0 // indirect
+	go.bug.st/relaxed-semver v0.15.0 // indirect
 	go.opentelemetry.io/auto/sdk v1.1.0 // indirect
 	go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.60.0 // indirect
 	go.opentelemetry.io/otel v1.35.0 // indirect
 	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.35.0 // indirect
 	go.opentelemetry.io/otel/metric v1.35.0 // indirect
 	go.opentelemetry.io/otel/trace v1.35.0 // indirect
-	golang.org/x/sys v0.31.0 // indirect
+	go.uber.org/atomic v1.9.0 // indirect
+	go.uber.org/multierr v1.9.0 // indirect
+	golang.org/x/crypto v0.37.0 // indirect
+	golang.org/x/net v0.39.0 // indirect
+	golang.org/x/sys v0.32.0 // indirect
+	golang.org/x/term v0.31.0 // indirect
+	golang.org/x/text v0.24.0 // indirect
 	golang.org/x/time v0.11.0 // indirect
+	google.golang.org/genproto/googleapis/rpc v0.0.0-20250218202821-56aae31c358a // indirect
+	google.golang.org/grpc v1.72.0 // indirect
+	google.golang.org/protobuf v1.36.6 // indirect
+	gopkg.in/warnings.v0 v0.1.2 // indirect
 	gotest.tools/v3 v3.5.2 // indirect
 )