@htlkg/data 0.0.3 → 0.0.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@htlkg/data",
-  "version": "0.0.3",
+  "version": "0.0.11",
   "type": "module",
   "exports": {
     ".": {
@@ -38,16 +38,19 @@
   ],
   "dependencies": {
     "@aws-amplify/api": "^6.0.0",
-    "vue": "^3.5.22",
     "zod": "^3.22.0",
-    "@htlkg/core": "0.0.3"
+    "@htlkg/core": "0.0.10"
+  },
+  "peerDependencies": {
+    "vue": "^3.5.0"
   },
   "publishConfig": {
-    "access": "restricted"
+    "access": "public"
   },
   "devDependencies": {
     "astro": "^5.14.7",
     "aws-amplify": "^6.15.7",
+    "vue": "^3.5.22",
     "tsup": "^8.0.0",
     "typescript": "^5.9.2",
     "vitest": "^3.2.4"
@@ -56,6 +59,9 @@
     "build": "tsup",
     "dev": "tsup --watch",
     "test": "vitest run",
-    "test:watch": "vitest"
+    "test:watch": "vitest",
+    "version:patch": "pnpm version patch --no-git-tag-version",
+    "version:minor": "pnpm version minor --no-git-tag-version",
+    "version:major": "pnpm version major --no-git-tag-version"
   }
 }

package/src/client/__tests__/server.test.ts

@@ -5,6 +5,12 @@ import type { ResourcesConfig } from 'aws-amplify';
 // Mock dependencies
 vi.mock('@htlkg/core/amplify-astro-adapter', () => ({
 	createRunWithAmplifyServerContext: vi.fn(() => vi.fn()),
+	createLogger: vi.fn(() => ({
+		debug: vi.fn(),
+		info: vi.fn(),
+		warn: vi.fn(),
+		error: vi.fn(),
+	})),
 }));
 
 vi.mock('aws-amplify/adapter-core/internals', () => ({

package/src/client/index.test.ts

@@ -36,7 +36,13 @@ vi.mock('aws-amplify/data', () => ({
 
 // Mock @htlkg/core/amplify-astro-adapter
 vi.mock('@htlkg/core/amplify-astro-adapter', () => ({
-	createRunWithAmplifyServerContext: vi.fn(() => vi.fn())
+	createRunWithAmplifyServerContext: vi.fn(() => vi.fn()),
+	createLogger: vi.fn(() => ({
+		debug: vi.fn(),
+		info: vi.fn(),
+		warn: vi.fn(),
+		error: vi.fn(),
+	})),
 }));
 
 // Mock aws-amplify/auth/server
@@ -103,113 +109,73 @@ describe('generateServerClient', () => {
 		vi.clearAllMocks();
 	});
 	
-	describe('with userPool auth mode (default)', () => {
-		it('should generate a server-side client with default userPool auth', () => {
-			const client = generateServerClient({
-				config: mockConfig
-			});
-			
+	describe('basic functionality', () => {
+		it('should generate a server-side client', () => {
+			const client = generateServerClient();
+
 			expect(client).toBeDefined();
 			expect(client.models).toBeDefined();
 		});
-		
-		it('should use userPool auth mode by default', () => {
-			generateServerClient({
-				config: mockConfig
-			});
-			
-			expect(generateDataClient).toHaveBeenCalledWith({
-				authMode: 'userPool'
-			});
+
+		it('should call generateDataClient without options', () => {
+			generateServerClient();
+
+			// generateServerClient calls generateDataClient without parameters
+			// authMode is specified per-operation, not at client creation
+			expect(generateDataClient).toHaveBeenCalled();
 		});
-		
+
 		it('should not call Amplify.configure (should be done at app startup)', () => {
 			vi.clearAllMocks();
-			
-			generateServerClient({
-				config: mockConfig
-			});
-			
+
+			generateServerClient();
+
 			// generateServerClient should NOT call Amplify.configure
 			// It should be called once at app startup (e.g., in amplify-server.ts)
 			expect(Amplify.configure).not.toHaveBeenCalled();
 		});
-		
+
 		it('should not call createRunWithAmplifyServerContext (should be done outside)', () => {
 			vi.clearAllMocks();
-			
-			generateServerClient({
-				config: mockConfig
-			});
-			
+
+			generateServerClient();
+
 			// generateServerClient should NOT call createRunWithAmplifyServerContext
 			// The caller should wrap the operation with runWithAmplifyServerContext
 			expect(createRunWithAmplifyServerContext).not.toHaveBeenCalled();
 		});
 	});
-	
-	describe('with apiKey auth mode', () => {
-		it('should generate a server-side client with API key auth', () => {
+
+	describe('with options', () => {
+		it('should accept authMode option', () => {
 			const client = generateServerClient({
-				config: mockConfig,
 				authMode: 'apiKey'
 			});
-			
+
 			expect(client).toBeDefined();
 			expect(client.models).toBeDefined();
 		});
-		
-		it('should use apiKey auth mode when specified', () => {
-			generateServerClient({
-				config: mockConfig,
-				authMode: 'apiKey'
-			});
-			
-			expect(generateDataClient).toHaveBeenCalledWith({
-				authMode: 'apiKey'
-			});
-		});
-	});
-	
-	describe('configuration', () => {
-		it('should accept valid Amplify configuration', () => {
-			const client = generateServerClient({
-				config: mockConfig
-			});
-			
-			expect(client).toBeDefined();
-		});
-		
-		it('should handle configuration with minimal auth setup', () => {
-			const minimalConfig = {
-				auth: {
-					user_pool_id: 'test-pool',
-					aws_region: 'us-east-1',
-					user_pool_client_id: 'test-client'
-				}
-			};
-			
+
+		it('should accept userPool authMode option', () => {
 			const client = generateServerClient({
-				config: minimalConfig,
-				authMode: 'apiKey'
+				authMode: 'userPool'
 			});
-			
+
 			expect(client).toBeDefined();
+			expect(client.models).toBeDefined();
 		});
 	});
 	
 	describe('multiple instances', () => {
 		it('should allow creating multiple server clients', () => {
 			const client1 = generateServerClient({
-				config: mockConfig,
 				authMode: 'userPool'
 			});
-			
+
 			const client2 = generateServerClient({
-				config: mockConfig,
 				authMode: 'apiKey'
 			});
-			
+
 			expect(client1).toBeDefined();
 			expect(client2).toBeDefined();
 		});