fix: scope.scope() replaces current scope instead of chaining

Author: antfubot

packages/devframe/src/client/scope.test.ts

@@ -57,6 +57,14 @@ describe('client.scope()', () => {
     expect(scoped.scope(null)).toBe(rpc)
   })
 
+  it('replaces the scope instead of chaining when calling scope() on a scoped client', () => {
+    const { rpc } = createMockClient()
+    rpc.scope = (ns?: string | null) => (ns ? createScopedClientContext(rpc, ns) : rpc) as any
+    const scope1 = rpc.scope('first')
+    const scope2 = scope1.scope('second')
+    expect(scope2.namespace).toBe('second')
+  })
+
   it('qualifies bare call/callEvent/callOptional method names', () => {
     const { rpc } = createMockClient()
     const scoped = createScopedClientContext(rpc, 'my-plugin')

packages/devframe/src/client/scope.ts

@@ -85,7 +85,10 @@ export interface DevframeScopedClientContext<NS extends string = string, Setting
   rpc: DevframeScopedClientRpc<NS>
   /** Persisted settings for this namespace (`global` + `project`). */
   settings: DevframeSettings<Settings>
-  /** Derive another scope, or pass `null` / `''` to get the base client. */
+  /**
+   * Return a new scoped client, replacing the current scope. Pass `null`
+   * or `''` to un-scope and get the base client.
+   */
   scope: DevframeRpcClient['scope']
 }
 

packages/devframe/src/node/__tests__/scope.test.ts

@@ -47,6 +47,13 @@ describe('ctx.scope()', () => {
     expect(scoped.scope(null)).toBe(ctx)
   })
 
+  it('replaces the scope instead of chaining when calling scope() on a scoped context', async () => {
+    const { ctx } = await createCtx()
+    const scope1 = ctx.scope('first')
+    const scope2 = scope1.scope('second')
+    expect(scope2.namespace).toBe('second')
+  })
+
   it('exposes the base context and read-only fields', async () => {
     const { ctx } = await createCtx()
     const scoped = ctx.scope('my-plugin')

packages/devframe/src/types/scope.ts

@@ -208,6 +208,9 @@ export interface DevframeScopedNodeContext<NS extends string = string, Settings
   views: DevframeViewHost
   diagnostics: DevframeDiagnosticsHost
   agent: DevframeAgentHost
-  /** Derive another scope, or pass `null` / `''` to get the base context. */
+  /**
+   * Return a new scoped context, replacing the current scope. Pass `null`
+   * or `''` to un-scope and get the base context.
+   */
   scope: DevframeNodeContext['scope']
 }