refactor: move plugins to plugins folder (#81)

Author: MarioCadenas

packages/appkit/src/index.ts

@@ -13,7 +13,6 @@ export type {
   StreamExecutionSettings,
 } from "shared";
 export { isSQLTypeMarker, sql } from "shared";
-export { analytics } from "./analytics";
 export { CacheManager } from "./cache";
 export { getExecutionContext } from "./context";
 export { createApp } from "./core";
@@ -31,7 +30,7 @@ export {
 } from "./errors";
 // Plugin authoring
 export { Plugin, toPlugin } from "./plugin";
-export { server } from "./server";
+export { analytics, server } from "./plugins";
 // Telemetry (for advanced custom telemetry)
 export {
   type Counter,

packages/appkit/src/plugin/dev-reader.ts

@@ -1,6 +1,6 @@
 import { randomUUID } from "node:crypto";
 import type { TunnelConnection } from "shared";
-import { isRemoteTunnelAllowedByEnv } from "@/server/remote-tunnel/gate";
+import { isRemoteTunnelAllowedByEnv } from "@/plugins/server/remote-tunnel/gate";
 import { TunnelError } from "../errors";
 import { createLogger } from "../logging/logger";
 

packages/appkit/src/plugin/tests/dev-reader.test.ts

@@ -3,7 +3,7 @@ import { beforeEach, describe, expect, test, vi } from "vitest";
 import { TunnelError } from "../../errors";
 
 // Mock the gate to allow remote tunnel in tests
-vi.mock("@/server/remote-tunnel/gate", () => ({
+vi.mock("@/plugins/server/remote-tunnel/gate", () => ({
   isRemoteTunnelAllowedByEnv: () => true,
 }));
 

…ckages/appkit/src/analytics/analytics.ts …ppkit/src/plugins/analytics/analytics.tspackages/appkit/src/analytics/analytics.ts renamed to packages/appkit/src/plugins/analytics/analytics.ts packages/appkit/src/analytics/analytics.ts renamed to packages/appkit/src/plugins/analytics/analytics.ts

@@ -6,14 +6,14 @@ import type {
   SQLTypeMarker,
   StreamExecutionSettings,
 } from "shared";
-import { SQLWarehouseConnector } from "../connectors";
+import { SQLWarehouseConnector } from "../../connectors";
 import {
   getCurrentUserId,
   getWarehouseId,
   getWorkspaceClient,
-} from "../context";
-import { createLogger } from "../logging/logger";
-import { Plugin, toPlugin } from "../plugin";
+} from "../../context";
+import { createLogger } from "../../logging/logger";
+import { Plugin, toPlugin } from "../../plugin";
 import { queryDefaults } from "./defaults";
 import { QueryProcessor } from "./query";
 import type {

packages/appkit/src/analytics/defaults.ts …appkit/src/plugins/analytics/defaults.tspackages/appkit/src/analytics/defaults.ts renamed to packages/appkit/src/plugins/analytics/defaults.ts packages/appkit/src/analytics/defaults.ts renamed to packages/appkit/src/plugins/analytics/defaults.ts

@@ -1,8 +1,8 @@
 import { createHash } from "node:crypto";
 import type { sql } from "@databricks/sdk-experimental";
 import { isSQLTypeMarker, type SQLTypeMarker, sql as sqlHelpers } from "shared";
-import { getWorkspaceId } from "../context";
-import { ValidationError } from "../errors";
+import { getWorkspaceId } from "../../context";
+import { ValidationError } from "../../errors";
 
 type SQLParameterValue = SQLTypeMarker | null | undefined;
 

packages/appkit/src/analytics/index.ts …es/appkit/src/plugins/analytics/index.tspackages/appkit/src/analytics/index.ts renamed to packages/appkit/src/plugins/analytics/index.ts packages/appkit/src/analytics/index.ts renamed to packages/appkit/src/plugins/analytics/index.ts

@@ -7,7 +7,7 @@ import {
 } from "@tools/test-helpers";
 import { sql } from "shared";
 import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
-import { ServiceContext } from "../../context/service-context";
+import { ServiceContext } from "../../../context/service-context";
 import { AnalyticsPlugin, analytics } from "../analytics";
 import type { IAnalyticsConfig } from "../types";
 
@@ -45,7 +45,7 @@ const { mockCacheStore, mockCacheInstance } = vi.hoisted(() => {
   return { mockCacheStore: store, mockCacheInstance: instance };
 });
 
-vi.mock("../../cache", () => ({
+vi.mock("../../../cache", () => ({
   CacheManager: {
     getInstanceSync: vi.fn(() => mockCacheInstance),
   },

packages/appkit/src/analytics/query.ts …es/appkit/src/plugins/analytics/query.tspackages/appkit/src/analytics/query.ts renamed to packages/appkit/src/plugins/analytics/query.ts packages/appkit/src/analytics/query.ts renamed to packages/appkit/src/plugins/analytics/query.ts

@@ -1,7 +1,7 @@
 import { mockServiceContext } from "@tools/test-helpers";
 import { sql } from "shared";
 import { afterEach, beforeEach, describe, expect, test } from "vitest";
-import { ServiceContext } from "../../context/service-context";
+import { ServiceContext } from "../../../context/service-context";
 import { QueryProcessor } from "../query";
 
 describe("QueryProcessor", () => {