@synopackageland/cli 0.7.0 → 0.7.1

package/dist/templates/_executor/tasks_dsm71_test.go

@@ -0,0 +1,114 @@
+package main
+
+import (
+	"context"
+	"os"
+	"path/filepath"
+	"strings"
+	"testing"
+)
+
+// dsm71Scheduler answers like DSM 7.1.1-42962 (vt-62): TaskScheduler stops at
+// list v2 (offset/limit) and get v3. t-0234 measured the refusals: a version
+// above maxVersion 3 returns 104; a method absent at an existing version, 103.
+type dsm71Scheduler struct {
+	recordingScheduler
+	refusal string
+}
+
+func (s *dsm71Scheduler) Exec(ctx context.Context, path string, args []string) ([]byte, error) {
+	api, method, version := schedulerCall(args)
+	s.calls = append(s.calls, api+" "+method+" v"+version)
+	if api == "SYNO.Core.TaskScheduler" {
+		if method == "list" && version == "3" && s.refusal != "" {
+			return []byte(`{"success":false,"error":{"code":` + s.refusal + `}}`), nil
+		}
+		supported := map[string]string{"list": "2", "get": "3", "delete": "2", "run": "2"}
+		if supported[method] != version {
+			if version > "3" {
+				return []byte(`{"success":false,"error":{"code":104}}`), nil
+			}
+			return []byte(`{"success":false,"error":{"code":103}}`), nil
+		}
+		if method == "list" && (!contains(args, "offset=0") || !contains(args, "limit=500")) {
+			return []byte(`{"success":false,"error":{"code":101}}`), nil
+		}
+	}
+	return s.schedulerFixture.Exec(ctx, path, args)
+}
+
+func purgeWith(t *testing.T, cmd Command) {
+	t.Helper()
+	inventory := taskInventory{Package: "TestApp", PolicyDigest: strings.Repeat("b", 64), Tasks: []approvedTask{{Kind: "boot", Name: "TestApp executor boot", ScriptDigest: canonicalTaskDigest(t, "boot")}, {Kind: "control", Name: "TestApp executor control", ScriptDigest: canonicalTaskDigest(t, "control")}}}
+	state := filepath.Join(t.TempDir(), "state")
+	socket := filepath.Join(t.TempDir(), "socket")
+	os.MkdirAll(filepath.Join(state, "runtime"), 0700)
+	os.MkdirAll(socket, 0700)
+	if err := purgeWithInventory(context.Background(), &liveDSM{Command: cmd}, "TestApp", inventory, state, filepath.Join(state, "runtime"), socket, func(string) {}); err != nil {
+		t.Fatal(err)
+	}
+}
+
+func TestPurgeFindsAndDeletesBothTasksOnDSM71(t *testing.T) {
+	cmd := &dsm71Scheduler{}
+	purgeWith(t, cmd)
+	if strings.Join(cmd.deleted, ",") != "SYNO.Core.EventScheduler,SYNO.Core.TaskScheduler" {
+		t.Fatal("DSM 7.1 tasks left behind", cmd.deleted, cmd.calls)
+	}
+}
+
+func TestSchedulerReadFallsBackOnlyOnUnsupportedVersion(t *testing.T) {
+	for _, code := range []string{"105", "119", "102", "3121"} {
+		t.Run(code, func(t *testing.T) {
+			cmd := &dsm71Scheduler{refusal: code}
+			purgeWith(t, cmd)
+			for _, call := range cmd.calls {
+				if call == "SYNO.Core.TaskScheduler list v2" || call == "SYNO.Core.TaskScheduler get v3" {
+					t.Fatal("fell back after code", code, cmd.calls)
+				}
+			}
+			if contains(cmd.deleted, "SYNO.Core.TaskScheduler") {
+				t.Fatal("deleted without exact readback", cmd.deleted)
+			}
+		})
+	}
+}
+
+func TestModernSchedulerNeverUsesLegacyVersions(t *testing.T) {
+	cmd := &recordingScheduler{}
+	purgeWith(t, cmd)
+	for _, call := range cmd.calls {
+		if call == "SYNO.Core.TaskScheduler list v2" || call == "SYNO.Core.TaskScheduler get v3" {
+			t.Fatal("modern DSM used a legacy version", cmd.calls)
+		}
+	}
+	if len(cmd.deleted) != 2 {
+		t.Fatal(cmd.deleted)
+	}
+}
+
+type recordingScheduler struct {
+	schedulerFixture
+	calls []string
+}
+
+func (s *recordingScheduler) Exec(ctx context.Context, path string, args []string) ([]byte, error) {
+	api, method, version := schedulerCall(args)
+	s.calls = append(s.calls, api+" "+method+" v"+version)
+	return s.schedulerFixture.Exec(ctx, path, args)
+}
+
+func schedulerCall(args []string) (api, method, version string) {
+	for _, a := range args {
+		k, v, _ := strings.Cut(a, "=")
+		switch k {
+		case "api":
+			api = v
+		case "method":
+			method = v
+		case "version":
+			version = v
+		}
+	}
+	return
+}

package/dist/templates/_executor/tasks_test.go

@@ -6,6 +6,7 @@ import (
 	"errors"
 	"os"
 	"path/filepath"
+	"regexp"
 	"strings"
 	"synopkgland/executor/internal/taskscript"
 	"testing"
@@ -14,7 +15,9 @@ import (
 type schedulerFixture struct {
 	changed, foreignOwner, duplicate bool
 	script                           string
-	deleted                          []string
+	// deleteReply replaces the delete envelope, as DSM sends it.
+	deleteReply string
+	deleted     []string
 }
 
 func (s *schedulerFixture) Exec(_ context.Context, _ string, args []string) ([]byte, error) {
@@ -48,6 +51,9 @@ func (s *schedulerFixture) Exec(_ context.Context, _ string, args []string) ([]b
 		if api == "SYNO.Core.TaskScheduler" && !contains(args, `tasks=[{"id":7,"real_owner":"root"}]`) {
 			return nil, errors.New("wrong delete reference")
 		}
+		if s.deleteReply != "" {
+			return []byte(s.deleteReply), nil
+		}
 		data = map[string]any{}
 	} else if api == "SYNO.Core.EventScheduler" {
 		data = map[string]any{"task_name": "TestApp executor boot", "owner": map[string]string{"0": owner}, "operation": script, "operation_type": "script", "event": "bootup", "enable": true}
@@ -164,3 +170,54 @@ func canonicalTaskDigest(t *testing.T, kind string) string {
 	t.Fatal("task kind")
 	return ""
 }
+
+// t-0229: preuninst took 15-23 s; time each scheduler phase and the file
+// removal separately, naming the package, so a device log shows which waits.
+func TestPurgeLogsPhaseTimingWithPackageName(t *testing.T) {
+	cmd := &schedulerFixture{}
+	inventory := taskInventory{Package: "TestApp", PolicyDigest: strings.Repeat("b", 64), Tasks: []approvedTask{{Kind: "boot", Name: "TestApp executor boot", ScriptDigest: canonicalTaskDigest(t, "boot")}, {Kind: "control", Name: "TestApp executor control", ScriptDigest: canonicalTaskDigest(t, "control")}}}
+	state := filepath.Join(t.TempDir(), "state")
+	socket := filepath.Join(t.TempDir(), "socket")
+	os.MkdirAll(filepath.Join(state, "runtime"), 0700)
+	os.MkdirAll(socket, 0700)
+	logs := []string{}
+	if err := purgeWithInventory(context.Background(), &liveDSM{Command: cmd}, "TestApp", inventory, state, filepath.Join(state, "runtime"), socket, func(s string) { logs = append(logs, s) }); err != nil {
+		t.Fatal(err)
+	}
+	task := regexp.MustCompile(`^task cleanup timing: package=TestApp kind=(boot|control) lookup_ms=\d+ delete_ms=\d+ verify_ms=\d+$`)
+	total := regexp.MustCompile(`^purge timing: package=TestApp tasks_ms=\d+ files_ms=\d+$`)
+	kinds := []string{}
+	for _, line := range logs {
+		if m := task.FindStringSubmatch(line); m != nil {
+			kinds = append(kinds, m[1])
+		}
+	}
+	if strings.Join(kinds, ",") != "boot,control" || len(logs) == 0 || !total.MatchString(logs[len(logs)-1]) {
+		t.Fatal(logs)
+	}
+}
+
+// t-0237: EventScheduler delete v1 and TaskScheduler delete v2 answer a success
+// without data; that is a confirmed delete, while a refusal stays unconfirmed.
+func TestPurgeConfirmsDatalessDeleteSuccess(t *testing.T) {
+	for reply, unconfirmed := range map[string]bool{
+		`{"success":true}`:                        false,
+		`{"httpd_restart":false,"success":true}`:  false,
+		`{"success":false,"error":{"code":4800}}`: true,
+		`{"success":true,"error":{"code":4800}}`:  true,
+	} {
+		cmd := &schedulerFixture{deleteReply: reply}
+		inventory := taskInventory{Package: "TestApp", PolicyDigest: strings.Repeat("b", 64), Tasks: []approvedTask{{Kind: "boot", Name: "TestApp executor boot", ScriptDigest: canonicalTaskDigest(t, "boot")}, {Kind: "control", Name: "TestApp executor control", ScriptDigest: canonicalTaskDigest(t, "control")}}}
+		logs := []string{}
+		purgeTasks(context.Background(), &liveDSM{Command: cmd}, "TestApp", inventory, func(s string) { logs = append(logs, s) })
+		found := 0
+		for _, line := range logs {
+			if strings.HasPrefix(line, "task cleanup unconfirmed: ") {
+				found++
+			}
+		}
+		if (found == 2) != unconfirmed || (found != 0 && found != 2) || len(cmd.deleted) != 2 {
+			t.Fatal(reply, logs)
+		}
+	}
+}

package/dist/templates/_executor/upgrade_watch_test.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"context"
+	"errors"
 	"net"
 	"os"
 	"path/filepath"
@@ -70,13 +71,20 @@ func TestUpgradeGapPreservesRuntimeAndResumesOnlyApprovedPins(t *testing.T) {
 						}
 						opened.Add(1)
 						return func() { listener.Close(); closed.Add(1) }, nil
-					}, func(ctx context.Context) bool { return samePackagePins(ctx, target, ed, pd) }, func() error {
+					}, func(ctx context.Context) bool {
+						// The production decision: only differing pins ask for approval.
+						pins := packagePins(ctx, target, info, func(os.FileInfo) bool { return true }, ed, pd)
+						return approvedPins(pins, func() error { return errors.New("no anchored upgrade here") }, func() error { return nil }, func(message string) {
+							if strings.Contains(message, "administrator approval") {
+								select {
+								case approval <- struct{}{}:
+								default:
+								}
+							}
+						})
+					}, func() error {
 						purged.Add(1)
 						return finishControl(state, filepath.Join(state, "runtime"), sockDir, true)
-					}, func(message string) {
-						if strings.Contains(message, "administrator approval") {
-							approval <- struct{}{}
-						}
 					})
 				}()
 				t.Cleanup(func() {

package/dist/templates/_gateway/go.mod

@@ -1,3 +1,5 @@
 module synopkgland-gateway
 
 go 1.24
+
+toolchain go1.26.9

package/dist/templates/native-go/backend/go.mod

@@ -2,7 +2,7 @@ module dsmapp
 
 go 1.25.0
 
-toolchain go1.26.8
+toolchain go1.26.9
 
 require github.com/godbus/dbus/v5 v5.2.3-0.20260924123156-0ea052abeb82
 

package/dist/templates/native-go/backend/internal/filestation/client.go

@@ -21,6 +21,9 @@ const MaxBytes = 512 * 1024
 const maxMetadata = 1024 * 1024
 const PageSize = 100
 
+// downloadPeek bounds how much of a DSM 7.1 download is inspected for an error envelope.
+const downloadPeek = 4096
+
 // Client has no cookie jar or credentials. The destination is startup-owned,
 // literal loopback only; proxies, redirects and automatic retries are disabled.
 type Client struct {
@@ -269,15 +272,48 @@ func (c *Client) DownloadStream(ctx context.Context, r *http.Request, p string)
 	if e != nil {
 		return Download{}, e
 	}
-	disposition, _, _ := mime.ParseMediaType(resp.Header.Get("Content-Disposition"))
-	if disposition != "attachment" {
+	body := resp.Body
+	raw := resp.Header.Values("Content-Disposition")
+	if len(raw) == 0 {
+		// DSM 7.1 sends no Content-Disposition at all. Only a short body that is
+		// a complete DSM envelope is an error; everything else is the file.
+		if body, e = unlessEnvelope(resp); e != nil {
+			return Download{}, e
+		}
+	} else if disposition, _, _ := mime.ParseMediaType(raw[0]); len(raw) != 1 || disposition != "attachment" {
 		e = decode(resp, nil)
 		if e == nil {
 			e = fail("unavailable")
 		}
 		return Download{}, e
 	}
-	return Download{Body: resp.Body, ContentType: resp.Header.Get("Content-Type"), ContentDisposition: resp.Header.Get("Content-Disposition"), ContentLength: resp.ContentLength}, nil
+	return Download{Body: body, ContentType: resp.Header.Get("Content-Type"), ContentDisposition: resp.Header.Get("Content-Disposition"), ContentLength: resp.ContentLength}, nil
+}
+
+// unlessEnvelope returns the whole body as a stream unless it fits in the peek
+// and decodes as a DSM envelope, which is refused and never becomes content.
+func unlessEnvelope(resp *http.Response) (io.ReadCloser, error) {
+	head, e := io.ReadAll(io.LimitReader(resp.Body, downloadPeek+1))
+	if e != nil {
+		resp.Body.Close()
+		return nil, fail("read_failed")
+	}
+	if len(head) <= downloadPeek {
+		var envelope struct {
+			Success *bool `json:"success"`
+		}
+		if json.Unmarshal(head, &envelope) == nil && envelope.Success != nil {
+			resp.Body = io.NopCloser(bytes.NewReader(head))
+			if e = decode(resp, nil); e == nil {
+				e = fail("unavailable")
+			}
+			return nil, e
+		}
+	}
+	return struct {
+		io.Reader
+		io.Closer
+	}{io.MultiReader(bytes.NewReader(head), resp.Body), resp.Body}, nil
 }
 
 // Read is the bounded package-private job import, using the same stream.

package/dist/templates/native-go/backend/internal/filestation/download_dsm71_test.go

@@ -0,0 +1,82 @@
+package filestation
+
+import (
+	"context"
+	"fmt"
+	"io"
+	"net/http"
+	"strings"
+	"testing"
+)
+
+// DSM 7.1.1-42962 (vt-62, t-0231) answers Download v2 mode=download with the
+// file bytes and no Content-Disposition header. Its Content-Type was not
+// recorded, so every plausible type is covered.
+func dsm71Download(contentType, body string) http.HandlerFunc {
+	return func(w http.ResponseWriter, r *http.Request) {
+		if contentType != "" {
+			w.Header().Set("Content-Type", contentType)
+		}
+		fmt.Fprint(w, body)
+	}
+}
+
+func TestDSM71DownloadWithoutDispositionReturnsFileBytes(t *testing.T) {
+	for _, tt := range []struct{ contentType, body string }{
+		{"text/plain", "t0231 fixture a.txt"},
+		{"application/octet-stream", "\x00\x01binary"},
+		{"", "no type"},
+		{"application/json", `{"name":"a user file"}`},
+		{"application/json", `[{"success":false}]`},
+		{"text/plain", strings.Repeat("y", downloadPeek+7)},
+	} {
+		t.Run(tt.contentType, func(t *testing.T) {
+			c := fakeClient(t, dsm71Download(tt.contentType, tt.body))
+			b, err := c.Read(context.Background(), sessionRequest(), "/home/a")
+			if err != nil || string(b) != tt.body {
+				t.Fatalf("%v %q", err, b)
+			}
+		})
+	}
+}
+
+func TestDSM71DownloadStreamsBeyondThePeek(t *testing.T) {
+	body := strings.Repeat("z", MaxBytes*3)
+	c := fakeClient(t, dsm71Download("application/octet-stream", body))
+	d, err := c.DownloadStream(context.Background(), sessionRequest(), "/home/a")
+	if err != nil {
+		t.Fatal(err)
+	}
+	defer d.Body.Close()
+	b, err := io.ReadAll(d.Body)
+	if err != nil || string(b) != body || d.ContentType != "application/octet-stream" {
+		t.Fatal(err, len(b))
+	}
+}
+
+func TestDSM71DownloadEnvelopeWithoutDispositionIsNeverContent(t *testing.T) {
+	for _, tt := range []struct{ contentType, body, want string }{
+		{"application/json; charset=utf-8", `{"success":false,"error":{"code":408}}`, "denied"},
+		{"text/plain", `{"success":false,"error":{"code":119}}`, "session_required"},
+		{"", ` {"error":{"code":407},"success":false}` + "\n", "denied"},
+		{"application/json", `{"success":true,"data":{}}`, "unavailable"},
+	} {
+		t.Run(tt.want, func(t *testing.T) {
+			c := fakeClient(t, dsm71Download(tt.contentType, tt.body))
+			b, err := c.Read(context.Background(), sessionRequest(), "/home/a")
+			if err == nil || len(b) != 0 || !strings.Contains(err.Error(), tt.want) {
+				t.Fatalf("%v %q", err, b)
+			}
+		})
+	}
+}
+
+func TestNonAttachmentDispositionStaysRefused(t *testing.T) {
+	c := fakeClient(t, func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Disposition", `inline; filename="a"`)
+		fmt.Fprint(w, "abc")
+	})
+	if b, err := c.Read(context.Background(), sessionRequest(), "/home/a"); err == nil || len(b) != 0 {
+		t.Fatal("inline disposition admitted")
+	}
+}

package/dist/templates/native-go/backend/internal/identity/README.md

@@ -16,7 +16,11 @@ The transport accepts same-origin JSON POST requests:
 - `identity.logout {}` removes the app session and clears the cookie.
 
 NormalUser.get is the only DSM cookie identity check; anonymous Initdata success
-cannot establish identity. Local `user.Lookup` binds the username to its UID.
+cannot establish identity. DSM web logins enable SynoToken, and DSM refuses
+their cookie alone (119), so the check also forwards the request's single
+`X-SYNO-TOKEN` header, which the DSM window's `ctx.call` sends. Sessions without
+SynoToken still verify with the cookie alone. The token is never put in a URL,
+logged or stored. Local `user.Lookup` binds the username to its UID.
 Auth login creates a temporary SID, then immediately logs it out before issuing
 an app session. A logout failure refuses the app login. No password, OTP, DSM SID
 or DSM cookie is logged or persisted. Only hashes of app bearer IDs are stored.

package/dist/templates/native-go/backend/internal/identity/dsm.go

@@ -9,6 +9,7 @@ import (
 	"net/url"
 	"strings"
 	"time"
+	"unicode"
 )
 
 type dsmReply struct {
@@ -25,7 +26,9 @@ type dsmReply struct {
 	} `json:"error"`
 }
 
-func (s *Service) dsmCall(ctx context.Context, form url.Values, cookie string) (dsmReply, error) {
+// dsmCall sends token only as X-SYNO-TOKEN: DSM web logins enable SynoToken,
+// and their cookie alone is refused with 119.
+func (s *Service) dsmCall(ctx context.Context, form url.Values, cookie, token string) (dsmReply, error) {
 	var out dsmReply
 	ctx, cancel := context.WithTimeout(ctx, 10*time.Second)
 	defer cancel()
@@ -37,6 +40,9 @@ func (s *Service) dsmCall(ctx context.Context, form url.Values, cookie string) (
 	if cookie != "" {
 		req.AddCookie(&http.Cookie{Name: "id", Value: cookie})
 	}
+	if token != "" {
+		req.Header.Set("X-SYNO-TOKEN", token)
+	}
 	res, err := s.cfg.Client.Do(req)
 	if err != nil {
 		return out, errors.New("DSM unavailable")
@@ -48,19 +54,32 @@ func (s *Service) dsmCall(ctx context.Context, form url.Values, cookie string) (
 	}
 	return out, nil
 }
-func (s *Service) verify(ctx context.Context, cookie string) (Subject, error) {
-	out, err := s.dsmCall(ctx, url.Values{"api": {"SYNO.Core.NormalUser"}, "method": {"get"}, "version": {"1"}}, cookie)
+
+// verify checks the caller's DSM session with the request's cookie and, when the
+// browser sent one, its SynoToken. Neither is stored or logged.
+func (s *Service) verify(r *http.Request, cookie string) (Subject, error) {
+	token := synoToken(r)
+	out, err := s.dsmCall(r.Context(), url.Values{"api": {"SYNO.Core.NormalUser"}, "method": {"get"}, "version": {"1"}}, cookie, token)
 	if err != nil || !out.Success {
 		return Subject{}, errors.New("DSM session refused")
 	}
 	return s.resolve(out.Data.Username)
 }
+
+// synoToken returns the single, well-formed X-SYNO-TOKEN header or "".
+func synoToken(r *http.Request) string {
+	values := r.Header.Values("X-SYNO-TOKEN")
+	if len(values) != 1 || len(values[0]) > 4096 || strings.ContainsFunc(values[0], unicode.IsControl) {
+		return ""
+	}
+	return values[0]
+}
 func (s *Service) login(ctx context.Context, account, password, otp string) (Subject, string) {
 	form := url.Values{"api": {"SYNO.API.Auth"}, "method": {"login"}, "version": {"7"}, "account": {account}, "passwd": {password}, "format": {"sid"}}
 	if otp != "" {
 		form.Set("otp_code", otp)
 	}
-	out, err := s.dsmCall(ctx, form, "")
+	out, err := s.dsmCall(ctx, form, "", "")
 	if err != nil {
 		return Subject{}, "auth_unavailable"
 	}
@@ -84,7 +103,7 @@ func (s *Service) login(ctx context.Context, account, password, otp string) (Sub
 	}
 	// The temporary DSM session exists only to prove identity. Dispose of it
 	// before issuing an application session, including when local lookup fails.
-	loggedOut, err := s.dsmCall(context.WithoutCancel(ctx), url.Values{"api": {"SYNO.API.Auth"}, "method": {"logout"}, "version": {"7"}, "_sid": {out.Data.SID}}, "")
+	loggedOut, err := s.dsmCall(context.WithoutCancel(ctx), url.Values{"api": {"SYNO.API.Auth"}, "method": {"logout"}, "version": {"7"}, "_sid": {out.Data.SID}}, "", "")
 	if err != nil || !loggedOut.Success {
 		return Subject{}, "auth_unavailable"
 	}

package/dist/templates/native-go/backend/internal/identity/http.go

@@ -237,7 +237,7 @@ func (s *Service) handle(w http.ResponseWriter, r *http.Request, req request) {
 					code = "session_required"
 					return nil
 				}
-				sub, err := s.verify(r.Context(), dc.Value)
+				sub, err := s.verify(r, dc.Value)
 				if err != nil {
 					code = "session_required"
 					return nil

package/dist/templates/native-go/backend/internal/identity/identity_test.go

@@ -27,6 +27,11 @@ type fixture struct {
 	logout  int
 	calls   []string
 	dir     string
+	// synoToken makes the fake DSM session a SynoToken session: NormalUser.get
+	// then needs the cookie and this X-SYNO-TOKEN header, as DSM web logins do.
+	synoToken string
+	// token is the X-SYNO-TOKEN the browser sends with each App request.
+	token string
 }
 
 func setup(t *testing.T) *fixture {
@@ -40,7 +45,10 @@ func setup(t *testing.T) *fixture {
 		switch name {
 		case "SYNO.Core.NormalUser.get":
 			c, err := r.Cookie("id")
-			if f.valid && err == nil && c.Value == "valid-dsm-cookie" {
+			if r.Form.Has("SynoToken") {
+				t.Error("SynoToken must travel only in the header")
+			}
+			if f.valid && err == nil && c.Value == "valid-dsm-cookie" && (f.synoToken == "" || r.Header.Get("X-SYNO-TOKEN") == f.synoToken) {
 				fmt.Fprint(w, `{"success":true,"data":{"username":"alice"}}`)
 			} else {
 				fmt.Fprint(w, `{"success":false,"error":{"code":119}}`)
@@ -93,6 +101,9 @@ func (f *fixture) call(method, params string, appCookie *http.Cookie, dsmCookie
 	if dsmCookie {
 		req.AddCookie(&http.Cookie{Name: "id", Value: "valid-dsm-cookie"})
 	}
+	if f.token != "" {
+		req.Header.Set("X-SYNO-TOKEN", f.token)
+	}
 	out := httptest.NewRecorder()
 	f.service.Wrap(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		id, err := f.service.Authenticate(r)
@@ -148,6 +159,42 @@ func TestCookieExchangeAndNoInitdataShortcut(t *testing.T) {
 		}
 	}
 }
+func TestSynoTokenSessionBootstrapsAndRevalidates(t *testing.T) {
+	f := setup(t)
+	f.synoToken = "secret-syno-token"
+	if _, out := f.call("identity.session", `{"bootstrap":true}`, nil, true); errorCode(out) != "session_required" {
+		t.Fatalf("cookie alone must not pass a SynoToken session: %v", out)
+	}
+	f.token = "secret-syno-token"
+	rec, out := f.call("identity.session", `{"bootstrap":true}`, nil, true)
+	if errorCode(out) != "" {
+		t.Fatalf("bootstrap with SynoToken: %v", out)
+	}
+	c := sessionCookie(t, rec)
+	if _, out = f.call("greeting.set", `{"text":"x"}`, c, true); errorCode(out) != "" {
+		t.Fatalf("live revalidation with SynoToken: %v", out)
+	}
+	filepath.WalkDir(f.dir, func(path string, d os.DirEntry, err error) error {
+		if err != nil {
+			t.Fatal(err)
+		}
+		if raw, _ := os.ReadFile(path); !d.IsDir() && strings.Contains(string(raw), "secret-syno-token") {
+			t.Fatalf("SynoToken written to %s", path)
+		}
+		return nil
+	})
+	f.token = ""
+	if _, out = f.call("greeting.set", `{"text":"x"}`, c, true); errorCode(out) != "session_required" {
+		t.Fatalf("revalidation without SynoToken: %v", out)
+	}
+}
+func TestTokenlessSessionStillVerifiesWhenBrowserSendsToken(t *testing.T) {
+	f := setup(t)
+	f.token = "unused-token"
+	if _, out := f.call("identity.session", `{"bootstrap":true}`, nil, true); errorCode(out) != "" {
+		t.Fatal(out)
+	}
+}
 func TestLoginOTPLogoutAndNoCredentialsOnDisk(t *testing.T) {
 	f := setup(t)
 	f.otp = true

package/dist/templates/native-go/backend/internal/identity/service.go

@@ -145,7 +145,7 @@ func (s *Service) current(st *state, r *http.Request, revalidate bool) (string,
 		if err != nil || dc.Value == "" {
 			return "", record{}, errRevalidation
 		}
-		live, err := s.verify(r.Context(), dc.Value)
+		live, err := s.verify(r, dc.Value)
 		if err != nil || live != sub {
 			delete(st.Sessions, key)
 			return "", record{}, api.ErrNoSession

package/dist/templates/native-go-delegate/frontend/test/delegate.test.ts

@@ -128,3 +128,18 @@ test('startup waiting catalog covers exactly nine distinct language messages',()
  expect(Object.keys(waitingMessages).sort()).toEqual(['en','zh-TW','zh-CN','de','es','fr','it','ja','ko'].sort())
  expect(new Set(Object.values(waitingMessages)).size).toBe(9)
 })
+test('a blocked approval shows the DSM reason, not only the generic unavailable text',async()=>{
+ const setup:ExecutorSetup={control:vi.fn(async()=>({state:'ready'})),status:vi.fn(async()=>({state:'needs-admin-approval'})),authorize:vi.fn(async()=>({state:'blocked',reason:'dsm_104: scheduler readback or operation failed.'}))}
+ const root=mount(ExecutorSettings,{admin:true},setup);await settle();const password=root.querySelector('input[type="password"]') as HTMLInputElement;password.value='once';password.dispatchEvent(new Event('input',{bubbles:true}));await settle();root.querySelector('form')!.dispatchEvent(new Event('submit',{bubbles:true,cancelable:true}));await settle()
+ expect(root.textContent).toContain('Background file access is unavailable.');expect(root.textContent).toContain('Details: dsm_104: scheduler readback or operation failed.')
+})
+test('a task that is only missing shows no extra details',async()=>{
+ const setup:ExecutorSetup={control:vi.fn(async()=>({state:'ready'})),status:vi.fn(async()=>({state:'needs-admin-approval',reason:'Task is missing.'})),authorize:vi.fn(async()=>({state:'ready'}))}
+ const root=mount(ExecutorSettings,{admin:true},setup);await settle();expect(root.textContent).not.toContain('Task is missing.')
+})
+test('settings follow an approval made from the window panel',async()=>{
+ let notify:((r:{state:'ready'})=>void)|undefined
+ const setup={control:vi.fn(async()=>({state:'ready'})),status:vi.fn(async()=>({state:'needs-admin-approval'})),authorize:vi.fn(async()=>({state:'ready'})),subscribe:(listener:any)=>{notify=listener}} as ExecutorSetup
+ const root=mount(ExecutorSettings,{admin:true},setup);await settle();expect(root.textContent).toContain('An administrator must approve')
+ notify!({state:'ready'});await settle();expect(root.textContent).toContain('Background file access is ready.')
+})

package/dist/templates/native-go-root-ops/backend/internal/app/executor.go

@@ -9,6 +9,8 @@ import (
 	"net/http"
 	"os"
 	"path/filepath"
+	"strings"
+	"syscall"
 	"time"
 
 	"dsmapp/internal/api"
@@ -51,8 +53,31 @@ func (cfg Config) executorControl(_ context.Context, caller api.Caller, raw json
 	return map[string]bool{"written": true}, nil
 }
 
+// The executor closes its socket while paused: the package is stopped, or an
+// upgrade changed the pins and awaits administrator approval.
 func (cfg Config) executorStatus(ctx context.Context, _ api.Caller, _ json.RawMessage) (any, error) {
-	return cfg.executorRequest(ctx, "status", nil, nil)
+	status, err := cfg.executorRequest(ctx, "status", nil, nil)
+	if errors.Is(err, syscall.ENOENT) || errors.Is(err, syscall.ECONNREFUSED) {
+		next := "Start the package."
+		if cfg.startRequested() {
+			next = "The package is running: an administrator approves background access again in Administrator setup; the executor log names the cause."
+		}
+		return map[string]any{"ready": false, "state": "paused", "reason": "executor_not_listening", "next": next}, nil
+	}
+	return status, err
+}
+
+// Package start and stop write the lifecycle mailbox; its last request tells a
+// stopped package from a running one whose executor stays paused.
+func (cfg Config) startRequested() bool {
+	f, err := os.Open(filepath.Join(cfg.DataDir, "executor", "control"))
+	if err != nil {
+		return false
+	}
+	defer f.Close()
+	b := make([]byte, 16)
+	n, _ := f.Read(b)
+	return strings.TrimSpace(string(b[:n])) == "start"
 }
 
 func (cfg Config) executorRootRun(ctx context.Context, caller api.Caller, req *http.Request, raw json.RawMessage) (any, error) {
@@ -73,8 +98,11 @@ func (cfg Config) executorRootRun(ctx context.Context, caller api.Caller, req *h
 	return cfg.executorRequest(ctx, "root.run", auth, map[string]string{"name": in.Name})
 }
 
+// executorSocketBase is a variable so tests can use a temporary directory.
+var executorSocketBase = "/run/synopkgland"
+
 func (cfg Config) executorRequest(ctx context.Context, op string, auth, params any) (any, error) {
-	conn, err := (&net.Dialer{Timeout: 5 * time.Second}).DialContext(ctx, "unix", "/run/synopkgland/"+cfg.PackageID+"/executor.sock")
+	conn, err := (&net.Dialer{Timeout: 5 * time.Second}).DialContext(ctx, "unix", filepath.Join(executorSocketBase, cfg.PackageID, "executor.sock"))
 	if err != nil {
 		return nil, err
 	}