Skip to content

Commit 0cc981d

Browse files
MarioCadenasIsaac
andcommitted
chore: merge execution-standalone-user into execution-identity-lifecycle
Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
2 parents 671b741 + e4b8fc2 commit 0cc981d

6 files changed

Lines changed: 2 additions & 12 deletions

File tree

‎packages/appkit/src/context/caller-context.ts‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,6 @@ export type CallerPrincipal = Readonly<{
99
userEmail?: string;
1010
}>;
1111

12-
/** @deprecated Use CallerPrincipal. Retained for backward compatibility. */
13-
export type Principal = CallerPrincipal;
14-
1512
/** Caller identity and workspace for one immutable execution scope. */
1613
export interface CallerContext {
1714
readonly client: ServiceContextState["client"];

‎packages/appkit/src/context/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ export {
1616
type CallerContext,
1717
type CallerPrincipal,
1818
type ExecutionContext,
19-
type Principal,
2019
isCallerContext,
2120
} from "./caller-context";
2221
export { ServiceContext } from "./service-context";

‎packages/appkit/src/context/scoped-api.ts‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import { isPlainObject } from "../utils/is-plain-object";
22
import type { RequestScope } from "./request-scope";
33

4-
export { isPlainObject } from "../utils/is-plain-object";
5-
64
const EXCLUDED_FROM_PROXY = new Set([
75
"setup",
86
"shutdown",

‎packages/appkit/src/core/agent/run-agent.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import {
1515
SUPERVISOR_EXTENSION_KEY,
1616
type SupervisorTool,
1717
} from "../../agents/supervisor-api";
18-
import { type Principal, runInCallerContext } from "../../context";
18+
import { type CallerPrincipal, runInCallerContext } from "../../context";
1919
import { getClientOptions } from "../../context/client-options";
2020
import { AuthenticationError, ConfigurationError } from "../../errors";
2121
import { createLogger } from "../../logging/logger";
@@ -53,7 +53,7 @@ export interface RunAgentInput {
5353
*/
5454
caller?: {
5555
readonly token: string;
56-
readonly principal: Principal;
56+
readonly principal: CallerPrincipal;
5757
readonly host: string;
5858
readonly workspaceId: string;
5959
};

‎packages/appkit/src/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@ export {
4343
type CallerContext,
4444
type CallerPrincipal,
4545
type ExecutionContext,
46-
type Principal,
4746
type UserContext,
4847
ServiceContext,
4948
getCallerContext,

‎packages/appkit/src/utils/tests/is-plain-object.test.ts‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
import { describe, expect, test } from "vitest";
22

3-
import { isPlainObject as scopedIsPlainObject } from "../../context/scoped-api";
43
import { isPlainObject as pluginIsPlainObject } from "../../plugin/plugin";
54
import { isPlainObject as crudIsPlainObject } from "../../plugins/database/crud/contract";
65
import { isPlainObject } from "../is-plain-object";
76

87
describe.each([
98
{ name: "utility", check: isPlainObject },
10-
{ name: "scoped API export", check: scopedIsPlainObject },
119
{ name: "plugin export", check: pluginIsPlainObject },
1210
{ name: "CRUD export", check: crudIsPlainObject },
1311
])("$name", ({ check }) => {
@@ -42,7 +40,6 @@ test("CRUD keeps its explicit array rejection regardless of the prototype", () =
4240
for (const prototype of [Object.prototype, null]) {
4341
const array = Object.setPrototypeOf([], prototype);
4442
expect(isPlainObject(array)).toBe(true);
45-
expect(scopedIsPlainObject(array)).toBe(true);
4643
expect(pluginIsPlainObject(array)).toBe(true);
4744
expect(crudIsPlainObject(array)).toBe(false);
4845
}

0 commit comments

Comments
 (0)