From 7cc9f2d142fac863eff173515f2a81e0a5c9e0f4 Mon Sep 17 00:00:00 2001 From: toki Date: Sun, 2 Aug 2026 21:12:23 +0900 Subject: [PATCH] =?UTF-8?q?refactor(execution):=20provider=20=EC=A0=84?= =?UTF-8?q?=EC=9A=A9=20=EC=86=8C=EC=9C=A0=EA=B6=8C=20=ED=91=9C=ED=98=84?= =?UTF-8?q?=EC=9C=BC=EB=A1=9C=20=EC=A0=95=EC=A0=95=ED=95=9C=EB=8B=A4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 승격 이후 경계 정정으로, doc.go/types.go/main.go의 잔여 standalone Agent·terminal 소유권 문구를 provider 전용·run 단위 표현으로 바꾸고, CLI 설명을 검증하는 Node 회귀 테스트를 추가한다. 승격·복구 증거는 그대로 보존한다. --- apps/node/cmd/node/main.go | 2 +- apps/node/cmd/node/main_test.go | 11 +++++++++++ packages/go/execution/doc.go | 6 +++--- packages/go/execution/types.go | 2 +- 4 files changed, 16 insertions(+), 5 deletions(-) diff --git a/apps/node/cmd/node/main.go b/apps/node/cmd/node/main.go index 9e01db80..6b85a9ad 100644 --- a/apps/node/cmd/node/main.go +++ b/apps/node/cmd/node/main.go @@ -26,7 +26,7 @@ func main() { func rootCmd() *cobra.Command { root := &cobra.Command{ Use: "node", - Short: "IOP Node Agent — runs adapter executions on this device", + Short: "IOP Node — runs provider executions on this device", } root.PersistentFlags().StringVarP(&cfgFile, "config", "c", "configs/node.yaml", "config file path") diff --git a/apps/node/cmd/node/main_test.go b/apps/node/cmd/node/main_test.go index 09979fd7..a7d215cc 100644 --- a/apps/node/cmd/node/main_test.go +++ b/apps/node/cmd/node/main_test.go @@ -45,6 +45,17 @@ func TestRootCmdIncludesOperationalCommands(t *testing.T) { } } +func TestRootCmdUsesProviderOnlyDescription(t *testing.T) { + short := rootCmd().Short + lower := strings.ToLower(short) + if !strings.Contains(lower, "provider") { + t.Errorf("root command Short must describe provider execution; got %q", short) + } + if strings.Contains(lower, "agent") { + t.Errorf("root command Short must not claim Agent ownership; got %q", short) + } +} + func TestVersionCmdPrintsVersion(t *testing.T) { root := rootCmd() var out bytes.Buffer diff --git a/packages/go/execution/doc.go b/packages/go/execution/doc.go index b3384caf..de353588 100644 --- a/packages/go/execution/doc.go +++ b/packages/go/execution/doc.go @@ -1,5 +1,5 @@ -// Package execution contains the host-neutral execution contract shared by -// IOP Node and standalone agent hosts. It owns provider lifecycle, streaming -// events, typed failures, registry lifecycle, and terminal session primitives; +// Package execution contains provider-only, host-neutral execution contracts +// shared by Node and Edge-facing provider transports. It owns provider +// lifecycle, streaming events, typed failures, and registry lifecycle; // host wire translation remains outside this package. package execution diff --git a/packages/go/execution/types.go b/packages/go/execution/types.go index 42ae2661..1f76a83b 100644 --- a/packages/go/execution/types.go +++ b/packages/go/execution/types.go @@ -12,7 +12,7 @@ import ( // Providers must not use it to identify, resume, or retain a process. const DefaultSessionID = "default" -// ErrRunCancelled is returned by adapters when a run is cancelled without terminating the session. +// ErrRunCancelled is returned by providers when a single run is cancelled. var ErrRunCancelled = errors.New("run cancelled") // ExecutionSpec is the resolved, policy-applied specification for a single run.