From f4e66afc80710560517da4aacdb72702e52d34c5 Mon Sep 17 00:00:00 2001 From: Jeffrey Smith Date: Tue, 31 Mar 2026 17:36:14 +0000 Subject: [PATCH] Fix CI: test manifests need tools/pipes/hooks for extension type ValidateManifest() now enforces type-specific constraints on bundled install too. Two tests used type "extension" without any tools/pipes/hooks, which is correctly rejected. Added hooks field to make them valid. Co-Authored-By: Claude Opus 4.6 (1M context) --- server/handlers/packages_bundled_test.go | 1 + server/handlers/upgrade_test.go | 1 + 2 files changed, 2 insertions(+) diff --git a/server/handlers/packages_bundled_test.go b/server/handlers/packages_bundled_test.go index 8fcb284..88162e1 100644 --- a/server/handlers/packages_bundled_test.go +++ b/server/handlers/packages_bundled_test.go @@ -171,6 +171,7 @@ func TestBundledInstall_DormantPackage(t *testing.T) { "id": "dormant-pkg", "title": "Dormant Package", "type": "extension", + "hooks": []string{"on_install"}, "requires": []string{"chat"}, }) diff --git a/server/handlers/upgrade_test.go b/server/handlers/upgrade_test.go index 1226168..82a2d72 100644 --- a/server/handlers/upgrade_test.go +++ b/server/handlers/upgrade_test.go @@ -451,6 +451,7 @@ func TestUpgrade_PackageDormantOnUnmetRequires(t *testing.T) { "title": "Future", "type": "extension", "version": "1.0.0", + "hooks": []string{"on_install"}, "requires": []string{"kernel>=99.0.0"}, })