e145d48f22
* feat: add plugin system with grpc runtime and native ui plugins - go plugin backends ship as release tarballs (per-platform binaries, checksums) that kandev installs from url/upload or filesystem sideload+sync, then spawns and supervises via hashicorp/go-plugin (unix socket / loopback tcp+automtls) - strict typed grpc contract (kandev.plugin.v1): DeliverEvent/InvokeTool/HandleWebhook plus a capability-gated host service (state, secrets, emit); author sdk in pkg/pluginsdk; packaging cli in cmd/plugin-pack - native frontend bundles: window.registerKandevPlugin + registry/host loader with dynamic routes, nav items, slots and ws handlers; plugins settings page with install dialog, sync button and enable/disable/uninstall - gated behind the plugins feature flag (prod off, dev/e2e on); covered by unit and integration tests plus playwright e2e against a real packaged fixture plugin Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): hydrate base app context on plugin routes Plugin-owned SPA routes classify as RouteUnknown, so the boot payload shipped only feature flags — no workspaces, workflows, or kanban context. Native plugin UI (e.g. host.ui.TaskCreateDialog) then had an empty store. Hydrate the same base context as the Home route for unknown SPA routes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * feat(plugins): expose TaskCreateDialog on host.ui Lets a native plugin open kandev's real create-task modal (prefilled via initialValues) instead of POSTing directly. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): harden filesystem/URL sinks against CodeQL findings Add sink-level guards on the plugin trust boundary: securejoin containment on archive extraction (zip-slip), plugin-id validation before every fs_store path, http/https scheme validation on install-from-URL, and non-interpolated ws-handler error logging. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): address review findings (races, resource caps, lifecycle) - serialize per-plugin startup to close a TOCTOU double-spawn race, and let a plugin restart after exhausting attempts - bound plugin startup with an explicit 30s timeout - reject backslash paths and cap decompressed bytes by actual size in pkgtar; only mark a package Signed when a verifier actually ran and passed - delete plugin_state on uninstall; validate webhook keys against the manifest and cap webhook body size; restart an active plugin when a new version installs Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): sanitize plugin-id path sinks for CodeQL Pin the plugin id to a single path segment with filepath.Base at recordPath/ configPath and pkgtar.Remove — a barrier the taint analysis recognizes. Behaviour-identical for valid ids (already validated by safePluginID); clears the go/path-injection alerts. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): inline plugin-id validation at path sinks for CodeQL filepath.Base does not strip '..', so it is not a path-injection barrier. Call the recognized safePluginID / id-pattern guard inline in Delete/GetConfig/ SetConfig and pkgtar.Remove — the same barrier that already cleared the Get sink. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): address code-review findings (backend) - add capability-gating tests for the Host RPC service (was untested) - close a stop-during-start orphan race so Disable/Uninstall can't leave a supervised subprocess behind - on failed upgrade, remove only the new version dir (not all versions + data); restart the prior version - validate manifest version and reject ids ending in .config (record-file alias) - make Deliverer.Refresh non-blocking and cancel in-flight delivery on worker stop - clamp out-of-range webhook status, skip-and-warn corrupt records in List, write records atomically, semver-aware sideload version pick Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): address code-review findings (frontend) - wrap plugin routes + settings routes in a shared PluginErrorBoundary so a throwing plugin page shows a fallback instead of white-screening the SPA - fix disable/re-enable in one session: keep the cached registration and re-run initialize instead of relying on a cached ES-module re-import - prefix root-relative bundle/style URLs with apiBaseUrl (split-origin/Tauri) - surface partial-install warnings via toast.warning instead of a bare success - dedupe/remove injected style links on unload; gate PluginNavItems on the flag Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(plugins): serialize per-plugin lifecycle + guard config writes - add a per-plugin keyed lock in Service so concurrent Enable/Disable/Install/ Uninstall/UpdateConfig for one id are serialized (no status-machine race) - take s.mu in SetConfig/GetConfig and write config atomically - bound each plugin initialize() with a timeout so one hung bundle can't stall the rest of the boot load Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * test(plugins): split service_test.go to satisfy file-length lint Move the install/URL test group into service_install_test.go; no logic change. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * ci: build and ship the e2e fixture plugin package to E2E shards The plugin E2E specs' global-setup requires apps/backend/.build/kandev-plugin-e2e-1.0.0.tar.gz. Build it in the E2E build job, upload it as an artifact, and download it in both the regular and container shard jobs. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Kandev Agent <agent@kandev.dev> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
309 lines
10 KiB
Go
309 lines
10 KiB
Go
package plugins
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"fmt"
|
|
"os"
|
|
"path/filepath"
|
|
"sort"
|
|
"strings"
|
|
"time"
|
|
|
|
"go.uber.org/zap"
|
|
|
|
"github.com/kandev/kandev/internal/plugins/manifest"
|
|
"github.com/kandev/kandev/internal/plugins/store"
|
|
)
|
|
|
|
// tarballSuffix identifies a dropped plugin package file directly under the
|
|
// plugins directory (see scanTarballs).
|
|
const tarballSuffix = ".tar.gz"
|
|
|
|
// Sync scans the plugins directory for filesystem-level changes an operator
|
|
// made outside the install/enable/disable/uninstall API — directory
|
|
// sideloads and dropped tarballs — and reconciles the registry with what
|
|
// actually exists on disk, per docs/specs/plugins/spec.md ("Filesystem
|
|
// sideloading & sync"):
|
|
//
|
|
// 1. Dir sideloads: an extracted <id>/<version>/manifest.yaml with no
|
|
// existing {id}.yml record is registered StatusDisabled (never
|
|
// auto-spawned — sideloads are unverified).
|
|
// 2. Dropped tarballs: any *.tar.gz file sitting directly in the plugins
|
|
// directory is run through the normal verified install pipeline
|
|
// (Service.Install); its file is deleted on success.
|
|
// 3. Missing installs: any registered record whose InstallPath no longer
|
|
// exists on disk is stopped (if running) and marked StatusError.
|
|
//
|
|
// Individual item failures never abort the rest of the scan; they are
|
|
// collected into SyncResult.Errors. Concurrent Sync calls are serialized by
|
|
// syncMu so they cannot race each other into a double install.
|
|
func (s *Service) Sync(ctx context.Context) (*SyncResult, error) {
|
|
return s.sync(ctx, true), nil
|
|
}
|
|
|
|
// bootScan runs the conservative subset of Sync at startup: dir sideloads
|
|
// (registered disabled, so nothing new is ever auto-spawned) and
|
|
// missing-install detection. Dropped tarballs are intentionally left alone
|
|
// at boot — installing and spawning an unverified binary as a side effect
|
|
// of starting up is out of scope; an operator triggers that via Sync (the
|
|
// UI's Sync button or POST /api/plugins/sync).
|
|
func (s *Service) bootScan(ctx context.Context) *SyncResult {
|
|
return s.sync(ctx, false)
|
|
}
|
|
|
|
// sync is Sync's and bootScan's shared implementation, parameterized on
|
|
// whether dropped tarballs are installed this run.
|
|
func (s *Service) sync(ctx context.Context, includeTarballs bool) *SyncResult {
|
|
s.syncMu.Lock()
|
|
defer s.syncMu.Unlock()
|
|
|
|
result := &SyncResult{}
|
|
if s.pluginsDir == "" {
|
|
return result
|
|
}
|
|
|
|
added, sideloadErrs := s.scanDirSideloads()
|
|
result.Added = added
|
|
result.Errors = append(result.Errors, sideloadErrs...)
|
|
|
|
if includeTarballs {
|
|
installed, tarballErrs := s.scanTarballs(ctx)
|
|
result.Installed = installed
|
|
result.Errors = append(result.Errors, tarballErrs...)
|
|
}
|
|
|
|
result.Missing = s.scanMissingInstalls()
|
|
|
|
s.notifyDeliverer()
|
|
return result
|
|
}
|
|
|
|
// scanDirSideloads finds every <pluginsDir>/<id>/<version>/manifest.yaml
|
|
// tree with no existing {id}.yml record and registers it StatusDisabled.
|
|
// When more than one unregistered version dir exists for the same id, the
|
|
// lexically greatest version is chosen and the rest are reported as skipped
|
|
// via the returned errs.
|
|
func (s *Service) scanDirSideloads() (added []string, errs []SyncError) {
|
|
entries, err := os.ReadDir(s.pluginsDir)
|
|
if err != nil {
|
|
if os.IsNotExist(err) {
|
|
return nil, nil
|
|
}
|
|
return nil, []SyncError{{Path: s.pluginsDir, Reason: err.Error()}}
|
|
}
|
|
|
|
for _, entry := range entries {
|
|
if !entry.IsDir() {
|
|
continue
|
|
}
|
|
id := entry.Name()
|
|
if _, ok := s.registry.Get(id); ok {
|
|
continue // already registered, not a sideload candidate
|
|
}
|
|
|
|
version, skipped, err := s.chooseSideloadVersion(id)
|
|
if err != nil {
|
|
errs = append(errs, SyncError{Path: filepath.Join(s.pluginsDir, id), Reason: err.Error()})
|
|
continue
|
|
}
|
|
if version == "" {
|
|
continue // no manifest.yaml found under any version dir
|
|
}
|
|
for _, skip := range skipped {
|
|
errs = append(errs, SyncError{
|
|
Path: filepath.Join(s.pluginsDir, id, skip),
|
|
Reason: fmt.Sprintf("skipped: version %q already selected for unregistered plugin %q", version, id),
|
|
})
|
|
}
|
|
|
|
if err := s.registerSideload(id, version); err != nil {
|
|
errs = append(errs, SyncError{Path: filepath.Join(s.pluginsDir, id, version), Reason: err.Error()})
|
|
continue
|
|
}
|
|
added = append(added, id)
|
|
}
|
|
return added, errs
|
|
}
|
|
|
|
// chooseSideloadVersion lists id's version directories under pluginsDir and
|
|
// returns the semver-greatest one (manifest.CompareVersions) containing a
|
|
// manifest.yaml, plus every other candidate (to be reported as skipped by
|
|
// the caller).
|
|
func (s *Service) chooseSideloadVersion(id string) (version string, skipped []string, err error) {
|
|
versionEntries, err := os.ReadDir(filepath.Join(s.pluginsDir, id))
|
|
if err != nil {
|
|
if os.IsNotExist(err) {
|
|
return "", nil, nil
|
|
}
|
|
return "", nil, err
|
|
}
|
|
|
|
var candidates []string
|
|
for _, v := range versionEntries {
|
|
if !v.IsDir() {
|
|
continue
|
|
}
|
|
manifestPath := filepath.Join(s.pluginsDir, id, v.Name(), manifestFileName)
|
|
if _, statErr := os.Stat(manifestPath); statErr != nil {
|
|
continue
|
|
}
|
|
candidates = append(candidates, v.Name())
|
|
}
|
|
if len(candidates) == 0 {
|
|
return "", nil, nil
|
|
}
|
|
sort.Slice(candidates, func(i, j int) bool {
|
|
return manifest.CompareVersions(candidates[i], candidates[j]) < 0
|
|
})
|
|
chosen := candidates[len(candidates)-1]
|
|
return chosen, candidates[:len(candidates)-1], nil
|
|
}
|
|
|
|
// manifestFileName mirrors pkgtar's own constant (kept local — pkgtar does
|
|
// not export it, and duplicating a single string literal here avoids
|
|
// exporting it just for this one read-only check).
|
|
const manifestFileName = "manifest.yaml"
|
|
|
|
// registerSideload parses+validates <pluginsDir>/<id>/<version>/manifest.yaml,
|
|
// requires it to be runtime-managed with a matching id, and persists a
|
|
// fresh StatusDisabled record for it.
|
|
func (s *Service) registerSideload(id, version string) error {
|
|
versionDir := filepath.Join(s.pluginsDir, id, version)
|
|
data, err := os.ReadFile(filepath.Join(versionDir, manifestFileName))
|
|
if err != nil {
|
|
return fmt.Errorf("read %s: %w", manifestFileName, err)
|
|
}
|
|
m, err := manifest.Parse(data)
|
|
if err != nil {
|
|
return fmt.Errorf("parse %s: %w", manifestFileName, err)
|
|
}
|
|
if err := m.Validate(); err != nil {
|
|
return fmt.Errorf("invalid %s: %w", manifestFileName, err)
|
|
}
|
|
if !m.IsManaged() {
|
|
return fmt.Errorf("%s is not runtime-managed (runtime.type must be \"binary\")", manifestFileName)
|
|
}
|
|
if m.ID != id {
|
|
return fmt.Errorf("manifest id %q does not match directory id %q", m.ID, id)
|
|
}
|
|
|
|
rec := &store.Record{
|
|
Manifest: *m,
|
|
Status: StatusDisabled,
|
|
InstallPath: versionDir,
|
|
Signed: false,
|
|
InstalledAt: time.Now().UTC(),
|
|
}
|
|
if err := s.store.Save(rec); err != nil {
|
|
return fmt.Errorf("persist sideloaded record: %w", err)
|
|
}
|
|
s.registry.Add(rec)
|
|
return nil
|
|
}
|
|
|
|
// scanTarballs installs every *.tar.gz file sitting directly under
|
|
// pluginsDir via the normal verified install pipeline (Service.Install),
|
|
// deleting each on success. A package that fails pkgtar's verify/validate
|
|
// pipeline is left in place and reported via errs.
|
|
func (s *Service) scanTarballs(ctx context.Context) (installed []string, errs []SyncError) {
|
|
entries, err := os.ReadDir(s.pluginsDir)
|
|
if err != nil {
|
|
if os.IsNotExist(err) {
|
|
return nil, nil
|
|
}
|
|
return nil, []SyncError{{Path: s.pluginsDir, Reason: err.Error()}}
|
|
}
|
|
|
|
for _, entry := range entries {
|
|
if entry.IsDir() || !strings.HasSuffix(entry.Name(), tarballSuffix) {
|
|
continue
|
|
}
|
|
path := filepath.Join(s.pluginsDir, entry.Name())
|
|
id, err := s.installTarball(ctx, path)
|
|
if err != nil {
|
|
errs = append(errs, SyncError{Path: path, Reason: err.Error()})
|
|
continue
|
|
}
|
|
installed = append(installed, id)
|
|
}
|
|
return installed, errs
|
|
}
|
|
|
|
// installTarball opens path and installs it via Service.Install. A
|
|
// non-nil rec (even alongside a non-nil activation-only err — see Install's
|
|
// doc comment) means the package itself was valid and extracted, so the
|
|
// tarball is deleted; a nil rec means pkgtar rejected the package, so the
|
|
// file is left in place for the operator to fix.
|
|
func (s *Service) installTarball(ctx context.Context, path string) (string, error) {
|
|
f, err := os.Open(path)
|
|
if err != nil {
|
|
return "", fmt.Errorf("open dropped package: %w", err)
|
|
}
|
|
defer func() { _ = f.Close() }()
|
|
|
|
rec, installErr := s.Install(ctx, f)
|
|
if rec == nil {
|
|
return "", installErr
|
|
}
|
|
if err := os.Remove(path); err != nil {
|
|
s.log.Warn("plugins: sync failed to remove installed tarball",
|
|
zap.String("path", path), zap.Error(err))
|
|
}
|
|
return rec.ID, nil
|
|
}
|
|
|
|
// scanMissingInstalls transitions every registered record whose
|
|
// InstallPath no longer exists on disk to StatusError (stopping its
|
|
// process first, if running), and returns their ids.
|
|
func (s *Service) scanMissingInstalls() []string {
|
|
var missing []string
|
|
for _, rec := range s.registry.List() {
|
|
if rec.InstallPath == "" {
|
|
continue
|
|
}
|
|
if _, err := os.Stat(rec.InstallPath); err == nil {
|
|
continue
|
|
}
|
|
if s.runtime != nil && s.runtime.Running(rec.ID) {
|
|
s.runtime.Stop(rec.ID)
|
|
}
|
|
s.markMissing(rec.ID)
|
|
missing = append(missing, rec.ID)
|
|
}
|
|
return missing
|
|
}
|
|
|
|
// markMissing transitions id to StatusError via the normal FSM transition,
|
|
// falling back to a direct (transition-check-bypassing) status write if the
|
|
// transition is not a legal single-hop edge — e.g. the record is already
|
|
// StatusError, which SetStatus rejects as a same-status "transition".
|
|
func (s *Service) markMissing(id string) {
|
|
err := s.SetStatus(id, StatusError)
|
|
if err == nil {
|
|
s.notifyDeliverer()
|
|
return
|
|
}
|
|
var invalidErr *ErrInvalidTransition
|
|
if !errors.As(err, &invalidErr) {
|
|
s.log.Warn("plugins: sync failed to mark missing install as error",
|
|
zap.String("plugin_id", id), zap.Error(err))
|
|
return
|
|
}
|
|
if err := s.setStatusUnchecked(id, StatusError); err != nil {
|
|
s.log.Warn("plugins: sync failed to force missing install to error",
|
|
zap.String("plugin_id", id), zap.Error(err))
|
|
}
|
|
}
|
|
|
|
// setStatusUnchecked directly writes status onto id's record in both the
|
|
// registry and the store, bypassing SetStatus's FSM transition check. Only
|
|
// used by markMissing's fallback path above.
|
|
func (s *Service) setStatusUnchecked(id string, status Status) error {
|
|
updated, ok := s.registry.SetStatus(id, status)
|
|
if !ok {
|
|
return store.ErrNotFound
|
|
}
|
|
return s.store.Save(updated)
|
|
}
|