pouch cred config (#856)

NiloCK · web-flow · commit 6ea301481743 · 2025-08-05T06:46:48.000-03:00
- **bump: version 0.1.11-23**
- **any &gt; unknown**
- **bump: version 0.1.11-24**
- **suppress debug logs...**
- **bump: version 0.1.11-25**
- **bump**
- **replace hard-coded cookie-based auth w/ dynamic...**
diff --git a/packages/cli/package.json b/packages/cli/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "type": "module",
   "description": "CLI scaffolding tool for vue-skuilder projects",
   "bin": {
diff --git a/packages/client/package.json b/packages/client/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "license": "MIT",
   "main": "dist/index.js",
   "module": "dist/index.esm.js",
diff --git a/packages/common-ui/package.json b/packages/common-ui/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "main": "./dist/common-ui.umd.js",
   "module": "./dist/common-ui.es.js",
   "types": "./dist/index.d.ts",
diff --git a/packages/common/package.json b/packages/common/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "type": "module",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
diff --git a/packages/common/src/logger.ts b/packages/common/src/logger.ts
@@ -1,17 +1,17 @@
 /**
  * Standard logger interface for vue-skuilder packages
- * 
+ *
  * This interface enables dependency injection of logging functionality,
  * allowing different runtime contexts to provide appropriate logger implementations:
  * - Node.js contexts can use Winston
  * - Browser contexts can use console wrappers
  * - Test contexts can use mock loggers
  */
 export interface SkLogger {
-  debug(message: string, ...args: any[]): void;
-  info(message: string, ...args: any[]): void;
-  warn(message: string, ...args: any[]): void;
-  error(message: string, ...args: any[]): void;
+  debug(message: string, ...args: unknown[]): void;
+  info(message: string, ...args: unknown[]): void;
+  warn(message: string, ...args: unknown[]): void;
+  error(message: string, ...args: unknown[]): void;
 }
 
 /**
@@ -29,8 +29,8 @@ export const noOpLogger: SkLogger = {
  * Uses console methods with appropriate ESLint suppressions
  */
 export const consoleLogger: SkLogger = {
-  debug: (message: string, ...args: any[]) => console.debug(message, ...args), // eslint-disable-line no-console
-  info: (message: string, ...args: any[]) => console.info(message, ...args),   // eslint-disable-line no-console
-  warn: (message: string, ...args: any[]) => console.warn(message, ...args),   // eslint-disable-line no-console
-  error: (message: string, ...args: any[]) => console.error(message, ...args), // eslint-disable-line no-console
-};
+  debug: (message: string, ...args: unknown[]) => console.debug(message, ...args), // eslint-disable-line no-console
+  info: (message: string, ...args: unknown[]) => console.info(message, ...args), // eslint-disable-line no-console
+  warn: (message: string, ...args: unknown[]) => console.warn(message, ...args), // eslint-disable-line no-console
+  error: (message: string, ...args: unknown[]) => console.error(message, ...args), // eslint-disable-line no-console
+};
diff --git a/packages/courseware/package.json b/packages/courseware/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "type": "module",
   "main": "./dist/index.cjs.js",
   "module": "./dist/index.mjs",
diff --git a/packages/db/package.json b/packages/db/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "description": "Database layer for vue-skuilder",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
diff --git a/packages/db/src/impl/couch/CouchDBSyncStrategy.ts b/packages/db/src/impl/couch/CouchDBSyncStrategy.ts
@@ -8,7 +8,7 @@ import type { SyncStrategy } from '../common/SyncStrategy';
 import type { AccountCreationResult, AuthenticationResult } from '../common/types';
 import { getLocalUserDB, hexEncode, updateGuestAccountExpirationDate } from '../common';
 import pouch from './pouchdb-setup';
-import { pouchDBincludeCredentialsConfig } from './index';
+import { createPouchDBConfig } from './index';
 import { getLoggedInUsername } from './auth';
 
 const log = (s: any) => {
@@ -207,7 +207,7 @@ export class CouchDBSyncStrategy implements SyncStrategy {
     // see: https://github.com/pouchdb-community/pouchdb-authentication/issues/239
     const ret = new pouch(
       ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-      pouchDBincludeCredentialsConfig
+      createPouchDBConfig()
     );
 
     if (guestAccount) {
diff --git a/packages/db/src/impl/couch/adminDB.ts b/packages/db/src/impl/couch/adminDB.ts
@@ -1,7 +1,7 @@
 import pouch from './pouchdb-setup';
 import { ENV } from '@db/factory';
 import {
-  pouchDBincludeCredentialsConfig,
+  createPouchDBConfig,
   getStartAndEndKeys,
   getCredentialledCourseConfig,
   updateCredentialledCourseConfig,
@@ -21,7 +21,7 @@ export class AdminDB implements AdminDBInterface {
     //     if the user is not an admin
     this.usersDB = new pouch(
       ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + '_users',
-      pouchDBincludeCredentialsConfig
+      createPouchDBConfig()
     );
   }
 
diff --git a/packages/db/src/impl/couch/classroomDB.ts b/packages/db/src/impl/couch/classroomDB.ts
@@ -11,7 +11,7 @@ import pouch from './pouchdb-setup';
 import {
   getCourseDB,
   getStartAndEndKeys,
-  pouchDBincludeCredentialsConfig,
+  createPouchDBConfig,
   REVIEW_TIME_FORMAT,
 } from '.';
 import { CourseDB, getTag } from './courseDB';
@@ -97,7 +97,7 @@ export class StudentClassroomDB
     const dbName = `classdb-student-${this._id}`;
     this._db = new pouch(
       ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-      pouchDBincludeCredentialsConfig
+      createPouchDBConfig()
     );
     try {
       const cfg = await this._db.get<ClassroomConfig>(CLASSROOM_CONFIG);
@@ -209,11 +209,11 @@ export class TeacherClassroomDB extends ClassroomDBBase implements TeacherClassr
     const stuDbName = `classdb-student-${this._id}`;
     this._db = new pouch(
       ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-      pouchDBincludeCredentialsConfig
+      createPouchDBConfig()
     );
     this._stuDb = new pouch(
       ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + stuDbName,
-      pouchDBincludeCredentialsConfig
+      createPouchDBConfig()
     );
     try {
       return this._db
@@ -297,7 +297,7 @@ export function getClassroomDB(classID: string, version: 'student' | 'teacher'):
 
   return new pouch(
     ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-    pouchDBincludeCredentialsConfig
+    createPouchDBConfig()
   );
 }
 
diff --git a/packages/db/src/impl/couch/courseAPI.ts b/packages/db/src/impl/couch/courseAPI.ts
@@ -1,5 +1,5 @@
 import pouch from './pouchdb-setup';
-import { pouchDBincludeCredentialsConfig } from '.';
+import { createPouchDBConfig } from '.';
 import { ENV } from '@db/factory';
 // import { DataShape } from '../..base-course/Interfaces/DataShape';
 import { NameSpacer, ShapeDescriptor } from '@vue-skuilder/common';
@@ -279,6 +279,6 @@ export function getCourseDB(courseID: string): PouchDB.Database {
   const dbName = `coursedb-${courseID}`;
   return new pouch(
     ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-    pouchDBincludeCredentialsConfig
+    createPouchDBConfig()
   );
 }
diff --git a/packages/db/src/impl/couch/index.ts b/packages/db/src/impl/couch/index.ts
@@ -38,26 +38,58 @@ export function hexEncode(str: string): string {
 
   return returnStr;
 }
-export const pouchDBincludeCredentialsConfig: PouchDB.Configuration.RemoteDatabaseConfiguration = {
+const pouchDBincludeCredentialsConfig: PouchDB.Configuration.RemoteDatabaseConfiguration = {
   fetch(url: string | Request, opts: RequestInit): Promise<Response> {
     opts.credentials = 'include';
 
     return (pouch as any).fetch(url, opts);
   },
 } as PouchDB.Configuration.RemoteDatabaseConfiguration;
 
+/**
+ * Creates PouchDB configuration with appropriate authentication method
+ * - Uses HTTP Basic Auth when credentials are available (Node.js/MCP)
+ * - Falls back to cookie auth for browser environments
+ */
+export function createPouchDBConfig(): PouchDB.Configuration.RemoteDatabaseConfiguration {
+  // Check if running in Node.js with explicit credentials
+  const hasExplicitCredentials = ENV.COUCHDB_USERNAME && ENV.COUCHDB_PASSWORD;
+  const isNodeEnvironment = typeof window === 'undefined';
+  
+  if (hasExplicitCredentials && isNodeEnvironment) {
+    // Use HTTP Basic Auth for Node.js environments (MCP server)
+    return {
+      fetch(url: string | Request, opts: RequestInit = {}): Promise<Response> {
+        const basicAuth = btoa(`${ENV.COUCHDB_USERNAME}:${ENV.COUCHDB_PASSWORD}`);
+        const headers = new Headers(opts.headers || {});
+        headers.set('Authorization', `Basic ${basicAuth}`);
+        
+        const newOpts = {
+          ...opts,
+          headers: headers
+        };
+        
+        return (pouch as any).fetch(url, newOpts);
+      }
+    } as PouchDB.Configuration.RemoteDatabaseConfiguration;
+  }
+  
+  // Use cookie-based auth for browser environments or when no explicit credentials
+  return pouchDBincludeCredentialsConfig;
+}
+
 function getCouchDB(dbName: string): PouchDB.Database {
   return new pouch(
     ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-    pouchDBincludeCredentialsConfig
+    createPouchDBConfig()
   );
 }
 
 export function getCourseDB(courseID: string): PouchDB.Database {
   // todo: keep a cache of opened courseDBs? need to benchmark this somehow
   return new pouch(
     ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + 'coursedb-' + courseID,
-    pouchDBincludeCredentialsConfig
+    createPouchDBConfig()
   );
 }
 
@@ -188,7 +220,7 @@ export function getCouchUserDB(username: string): PouchDB.Database {
   // see: https://github.com/pouchdb-community/pouchdb-authentication/issues/239
   const ret = new pouch(
     ENV.COUCHDB_SERVER_PROTOCOL + '://' + ENV.COUCHDB_SERVER_URL + dbName,
-    pouchDBincludeCredentialsConfig
+    createPouchDBConfig()
   );
   if (guestAccount) {
     updateGuestAccountExpirationDate(ret);
diff --git a/packages/edit-ui/package.json b/packages/edit-ui/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "main": "./dist/edit-ui.umd.js",
   "module": "./dist/edit-ui.es.js",
   "types": "./dist/index.d.ts",
diff --git a/packages/express/package.json b/packages/express/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "description": "an API",
   "main": "dist/index.js",
   "type": "module",
diff --git a/packages/express/src/client-requests/pack-requests.ts b/packages/express/src/client-requests/pack-requests.ts
@@ -144,11 +144,11 @@ export async function packCourse(data: PackCourseData): Promise<PackCourseRespon
     
     // Use regular PouchDB for simple data reading
     logger.info(`Creating PouchDB instance with URL: ${courseDbUrl}`);
-    logger.info(`PouchDB constructor available: ${typeof PouchDB}`);
-    logger.info(`PouchDB adapters: ${JSON.stringify(Object.keys((PouchDB as any).adapters || {}))}`);
+    // logger.info(`PouchDB constructor available: ${typeof PouchDB}`);
+    // logger.info(`PouchDB adapters: ${JSON.stringify(Object.keys((PouchDB as any).adapters || {}))}`);
     
     const courseDb = new PouchDB(courseDbUrl);
-    logger.info(`PouchDB instance created, adapter: ${(courseDb as any).adapter}`);
+    // logger.info(`PouchDB instance created, adapter: ${(courseDb as any).adapter}`);
     
     const packResult = await packer.packCourseToFiles(courseDb, data.courseId, outputPath, fsAdapter);
     
diff --git a/packages/mcp/package.json b/packages/mcp/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@vue-skuilder/mcp",
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "type": "module",
   "exports": {
     ".": {
diff --git a/packages/platform-ui/package.json b/packages/platform-ui/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "scripts": {
     "dev": "vite",
     "build": "vite build",
diff --git a/packages/standalone-ui/package.json b/packages/standalone-ui/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "type": "module",
   "main": "./dist-lib/questions.cjs.js",
   "module": "./dist-lib/questions.mjs",
diff --git a/packages/studio-ui/package.json b/packages/studio-ui/package.json
@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.11-22",
+  "version": "0.1.11-25",
   "type": "module",
   "scripts": {
     "dev": "vite",
