@harborclient/team-hub 0.4.7 → 0.4.9

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## Unreleased
 
+## 0.4.8 - 2026-07-28
+
+- Enhance environment variable management and UI readiness. (`abab021c`)
+- Update variable schema and enhance environment management. (`34cc08f0`)
+
 ## 0.4.6 - 2026-07-27
 
 - Add sidebar marker support across collections, environments, folders, requests, and documents. (`872135d`)

package/dist/cli.js

@@ -569,7 +569,8 @@ function mapFirestoreEnvironment(id, data) {
     createdByUserId: data.createdByUserId ?? null,
     updatedByUserId: data.updatedByUserId ?? null,
     deletionLocked: data.deletionLocked ?? false,
-    marker: readSidebarMarker(data.marker)
+    marker: readSidebarMarker(data.marker),
+    parentUuid: data.parentUuid?.trim() || null
   };
 }
 function mapFirestoreSnippet(id, data) {
@@ -822,6 +823,10 @@ function normalizeVariable(value) {
     key: typeof value.key === "string" ? value.key : "",
     value: typeof value.value === "string" ? value.value : "",
     defaultValue: typeof value.defaultValue === "string" ? value.defaultValue : "",
+    /**
+     * Legacy rows without `enabled` remain active so existing data keeps working.
+     */
+    enabled: value.enabled !== false,
     share: typeof value.share === "boolean" ? value.share : false
   };
 }
@@ -1609,18 +1614,21 @@ var FirestoreDatabase = class _FirestoreDatabase {
       updatedAt: now,
       createdByUserId: actingUserId,
       updatedByUserId: actingUserId,
-      deletionLocked: false
+      deletionLocked: false,
+      parentUuid: null
     };
     await this.requireClient().collection(ENVIRONMENTS_COLLECTION).doc(id).set(data);
     await this.recordAuditEntry(actingUserId, "create", "environment", id);
     return mapFirestoreEnvironment(id, data);
   }
   /**
-   * Updates an environment's name and variables.
+   * Updates an environment's name, variables, and optional parent link.
    *
    * @param actingUserId - User performing the update action.
+   * @param marker - Optional sidebar marker; omit to leave unchanged.
+   * @param parentUuid - Parent environment id; `null` clears; omit to leave unchanged.
    */
-  async updateEnvironment(id, name, variables, actingUserId, marker) {
+  async updateEnvironment(id, name, variables, actingUserId, marker, parentUuid) {
     const trimmedName = trimRequiredName(name, "Environment name");
     const updatedAt = /* @__PURE__ */ new Date();
     const docRef = this.requireClient().collection(ENVIRONMENTS_COLLECTION).doc(id);
@@ -1630,33 +1638,43 @@ var FirestoreDatabase = class _FirestoreDatabase {
     }
     const existing = snapshot.data();
     const serializedMarker = marker !== void 0 ? serializeSidebarMarker(marker) : existing.marker;
+    const normalizedParent = parentUuid === void 0 ? void 0 : parentUuid?.trim() || null;
     const updated = {
       ...existing,
       name: trimmedName,
       variables,
       updatedAt,
       updatedByUserId: actingUserId,
-      ...marker !== void 0 ? { marker: serializedMarker } : {}
+      ...marker !== void 0 ? { marker: serializedMarker } : {},
+      ...normalizedParent === void 0 ? {} : { parentUuid: normalizedParent }
     };
     await docRef.update({
       name: trimmedName,
       variables,
       updatedAt,
       updatedByUserId: actingUserId,
-      ...marker !== void 0 ? { marker: serializedMarker } : {}
+      ...marker !== void 0 ? { marker: serializedMarker } : {},
+      ...normalizedParent === void 0 ? {} : { parentUuid: normalizedParent }
     });
     await this.recordAuditEntry(actingUserId, "update", "environment", id);
     return mapFirestoreEnvironment(id, updated);
   }
   /**
-   * Deletes an environment.
+   * Deletes an environment and orphans any direct children (clears their parentUuid).
    *
    * @param id - Environment ID to delete.
    * @param actingUserId - User performing the delete action.
    */
   async deleteEnvironment(id, actingUserId) {
     await this.recordAuditEntry(actingUserId, "delete", "environment", id);
-    await this.requireClient().collection(ENVIRONMENTS_COLLECTION).doc(id).delete();
+    const client = this.requireClient();
+    const children = await client.collection(ENVIRONMENTS_COLLECTION).where("parentUuid", "==", id).get();
+    const batch = client.batch();
+    for (const child of children.docs) {
+      batch.update(child.ref, { parentUuid: null });
+    }
+    batch.delete(client.collection(ENVIRONMENTS_COLLECTION).doc(id));
+    await batch.commit();
   }
   /**
    * Finds an environment by stable identifier.
@@ -2861,7 +2879,8 @@ function mapEnvironmentSqlRow(row) {
     createdByUserId: row.created_by_user_id ?? null,
     updatedByUserId: row.updated_by_user_id ?? null,
     deletionLocked: Boolean(row.deletion_locked),
-    marker: readSidebarMarker(row.marker)
+    marker: readSidebarMarker(row.marker),
+    parentUuid: row.parent_uuid?.trim() || null
   };
 }
 function parseSnippetScope(value) {
@@ -2999,8 +3018,10 @@ CREATE TABLE IF NOT EXISTS environments (
   updated_at DATETIME NOT NULL,
   created_by_user_id VARCHAR(36) NULL,
   updated_by_user_id VARCHAR(36) NULL,
+  parent_uuid VARCHAR(36) NULL,
   FOREIGN KEY (created_by_user_id) REFERENCES users(id) ON DELETE SET NULL,
-  FOREIGN KEY (updated_by_user_id) REFERENCES users(id) ON DELETE SET NULL
+  FOREIGN KEY (updated_by_user_id) REFERENCES users(id) ON DELETE SET NULL,
+  FOREIGN KEY (parent_uuid) REFERENCES environments(id) ON DELETE SET NULL
 )
 `.trim();
 var SNIPPETS_MIGRATION_SQL = `
@@ -3140,6 +3161,10 @@ var FOLDERS_PARENT_MIGRATION_SQL = `
 ALTER TABLE folders
   ADD COLUMN IF NOT EXISTS parent_folder_id VARCHAR(36) NULL
 `.trim();
+var ENVIRONMENTS_PARENT_UUID_MIGRATION_SQL = `
+ALTER TABLE environments
+  ADD COLUMN IF NOT EXISTS parent_uuid VARCHAR(36) NULL
+`.trim();
 var REQUESTS_ATTRIBUTION_MIGRATION_SQL = `
 ALTER TABLE requests
   ADD COLUMN IF NOT EXISTS created_by_user_id VARCHAR(36) NULL,
@@ -3295,7 +3320,8 @@ var MYSQL_MIGRATIONS = [
   USERS_SNIPPET_ACCESS_BACKFILL_SQL,
   RUN_RESULTS_MIGRATION_SQL,
   USER_INVITATIONS_MIGRATION_SQL,
-  ...MARKER_MIGRATIONS_SQL
+  ...MARKER_MIGRATIONS_SQL,
+  ENVIRONMENTS_PARENT_UUID_MIGRATION_SQL
 ];
 
 // src/db/mysql/schemas.ts
@@ -3354,7 +3380,7 @@ function serializeAccessList(access) {
 }
 var USER_SELECT_COLUMNS = `id, name, role, collection_access, environment_access, snippet_access, llm_access, llm_models, llm_monthly_token_limit, created_at, updated_at, created_by_user_id, updated_by_user_id`;
 var COLLECTION_SELECT_COLUMNS = `id, name, variables, headers, auth, pre_request_script, post_request_script, created_at, updated_at, created_by_user_id, updated_by_user_id, deletion_locked, marker`;
-var ENVIRONMENT_SELECT_COLUMNS = `id, name, variables, created_at, updated_at, created_by_user_id, updated_by_user_id, deletion_locked, marker`;
+var ENVIRONMENT_SELECT_COLUMNS = `id, name, variables, created_at, updated_at, created_by_user_id, updated_by_user_id, deletion_locked, marker, parent_uuid`;
 var SNIPPET_SELECT_COLUMNS = `id, name, code, scope, sort_order, created_at, updated_at, created_by_user_id, updated_by_user_id, deletion_locked`;
 var FOLDER_SELECT_COLUMNS = `id, collection_id, parent_folder_id, name, sort_order, created_at, updated_at, created_by_user_id, updated_by_user_id, marker`;
 var REQUEST_SELECT_COLUMNS = `id, collection_id, folder_id, name, method, url, headers, params, auth, body, body_type, pre_request_script, post_request_script, comment, sort_order, created_at, updated_at, created_by_user_id, updated_by_user_id, marker`;
@@ -4359,37 +4385,36 @@ var MysqlDatabase = class _MysqlDatabase {
     return mapEnvironmentSqlRow(row);
   }
   /**
-   * Updates an environment's name and variables.
+   * Updates an environment's name, variables, and optional parent link.
    *
    * @param actingUserId - User performing the update action.
+   * @param marker - Optional sidebar marker; omit to leave unchanged.
+   * @param parentUuid - Parent environment id; `null` clears; omit to leave unchanged.
    */
-  async updateEnvironment(id, name, variables, actingUserId, marker) {
+  async updateEnvironment(id, name, variables, actingUserId, marker, parentUuid) {
     const trimmedName = trimRequiredName(name, "Environment name");
     const updatedAt = /* @__PURE__ */ new Date();
-    const result = marker !== void 0 ? await this.executeStatement(
+    const setClauses = ["name = ?", "variables = ?", "updated_at = ?", "updated_by_user_id = ?"];
+    const params = [
+      trimmedName,
+      JSON.stringify(variables),
+      updatedAt,
+      actingUserId
+    ];
+    if (marker !== void 0) {
+      setClauses.push("marker = ?");
+      params.push(serializeSidebarMarker(marker));
+    }
+    if (parentUuid !== void 0) {
+      setClauses.push("parent_uuid = ?");
+      params.push(parentUuid?.trim() || null);
+    }
+    params.push(id);
+    const result = await this.executeStatement(
       `UPDATE environments
-      SET name = ?,
-        variables = ?,
-        updated_at = ?,
-        updated_by_user_id = ?,
-        marker = ?
+      SET ${setClauses.join(",\n        ")}
       WHERE id = ?`,
-      [
-        trimmedName,
-        JSON.stringify(variables),
-        updatedAt,
-        actingUserId,
-        serializeSidebarMarker(marker),
-        id
-      ]
-    ) : await this.executeStatement(
-      `UPDATE environments
-      SET name = ?,
-        variables = ?,
-        updated_at = ?,
-        updated_by_user_id = ?
-      WHERE id = ?`,
-      [trimmedName, JSON.stringify(variables), updatedAt, actingUserId, id]
+      params
     );
     if ((result.affectedRows ?? 0) === 0) {
       throw new Error("Environment not found");
@@ -4406,13 +4431,17 @@ var MysqlDatabase = class _MysqlDatabase {
     return mapEnvironmentSqlRow(row);
   }
   /**
-   * Deletes an environment.
+   * Deletes an environment and orphans any direct children (clears their parent_uuid).
    *
    * @param id - Environment ID to delete.
    * @param actingUserId - User performing the delete action.
    */
   async deleteEnvironment(id, actingUserId) {
     await this.recordAuditEntry(actingUserId, "delete", "environment", id);
+    await this.executeStatement(
+      "UPDATE environments SET parent_uuid = NULL WHERE parent_uuid = ?",
+      [id]
+    );
     await this.executeStatement("DELETE FROM environments WHERE id = ?", [id]);
   }
   /**
@@ -5756,7 +5785,8 @@ CREATE TABLE IF NOT EXISTS environments (
   created_at TIMESTAMPTZ NOT NULL,
   updated_at TIMESTAMPTZ NOT NULL,
   created_by_user_id TEXT REFERENCES users(id) ON DELETE SET NULL,
-  updated_by_user_id TEXT REFERENCES users(id) ON DELETE SET NULL
+  updated_by_user_id TEXT REFERENCES users(id) ON DELETE SET NULL,
+  parent_uuid TEXT REFERENCES environments(id) ON DELETE SET NULL
 );
 `.trim();
 var SNIPPETS_MIGRATION_SQL2 = `
@@ -6022,6 +6052,10 @@ var FOLDERS_MARKER_MIGRATION_SQL = buildMarkerMigrationSql("folders");
 var REQUESTS_MARKER_MIGRATION_SQL = buildMarkerMigrationSql("requests");
 var DOCUMENTS_MARKER_MIGRATION_SQL = buildMarkerMigrationSql("documents");
 var ENVIRONMENTS_MARKER_MIGRATION_SQL = buildMarkerMigrationSql("environments");
+var ENVIRONMENTS_PARENT_UUID_MIGRATION_SQL2 = `
+ALTER TABLE environments
+  ADD COLUMN IF NOT EXISTS parent_uuid TEXT REFERENCES environments(id) ON DELETE SET NULL;
+`.trim();
 var POSTGRES_MIGRATIONS = [
   USERS_MIGRATION_SQL2,
   API_TOKENS_MIGRATION_SQL2,
@@ -6056,7 +6090,8 @@ var POSTGRES_MIGRATIONS = [
   FOLDERS_MARKER_MIGRATION_SQL,
   REQUESTS_MARKER_MIGRATION_SQL,
   DOCUMENTS_MARKER_MIGRATION_SQL,
-  ENVIRONMENTS_MARKER_MIGRATION_SQL
+  ENVIRONMENTS_MARKER_MIGRATION_SQL,
+  ENVIRONMENTS_PARENT_UUID_MIGRATION_SQL2
 ];
 
 // src/db/postgres/schemas.ts
@@ -7010,37 +7045,36 @@ var PostgresDatabase = class _PostgresDatabase {
     return mapEnvironmentSqlRow(row);
   }
   /**
-   * Updates an environment's name and variables.
+   * Updates an environment's name, variables, and optional parent link.
    *
    * @param actingUserId - User performing the update action.
+   * @param marker - Optional sidebar marker; omit to leave unchanged.
+   * @param parentUuid - Parent environment id; `null` clears; omit to leave unchanged.
    */
-  async updateEnvironment(id, name, variables, actingUserId, marker) {
+  async updateEnvironment(id, name, variables, actingUserId, marker, parentUuid) {
     const trimmedName = trimRequiredName(name, "Environment name");
     const updatedAt = /* @__PURE__ */ new Date();
-    const result = marker !== void 0 ? await this.query(
-      `UPDATE environments
-      SET name = $1,
-        variables = $2,
-        updated_at = $3,
-        updated_by_user_id = $4,
-        marker = $5
-      WHERE id = $6`,
-      [
-        trimmedName,
-        JSON.stringify(variables),
-        updatedAt,
-        actingUserId,
-        serializeSidebarMarker(marker),
-        id
-      ]
-    ) : await this.query(
+    const setClauses = [
+      "name = $1",
+      "variables = $2",
+      "updated_at = $3",
+      "updated_by_user_id = $4"
+    ];
+    const params = [trimmedName, JSON.stringify(variables), updatedAt, actingUserId];
+    if (marker !== void 0) {
+      params.push(serializeSidebarMarker(marker));
+      setClauses.push(`marker = $${params.length}`);
+    }
+    if (parentUuid !== void 0) {
+      params.push(parentUuid?.trim() || null);
+      setClauses.push(`parent_uuid = $${params.length}`);
+    }
+    params.push(id);
+    const result = await this.query(
       `UPDATE environments
-      SET name = $1,
-        variables = $2,
-        updated_at = $3,
-        updated_by_user_id = $4
-      WHERE id = $5`,
-      [trimmedName, JSON.stringify(variables), updatedAt, actingUserId, id]
+      SET ${setClauses.join(",\n        ")}
+      WHERE id = $${params.length}`,
+      params
     );
     if ((result.rowCount ?? 0) === 0) {
       throw new Error("Environment not found");
@@ -7057,13 +7091,14 @@ var PostgresDatabase = class _PostgresDatabase {
     return mapEnvironmentSqlRow(row);
   }
   /**
-   * Deletes an environment.
+   * Deletes an environment and orphans any direct children (clears their parent_uuid).
    *
    * @param id - Environment ID to delete.
    * @param actingUserId - User performing the delete action.
    */
   async deleteEnvironment(id, actingUserId) {
     await this.recordAuditEntry(actingUserId, "delete", "environment", id);
+    await this.query("UPDATE environments SET parent_uuid = NULL WHERE parent_uuid = $1", [id]);
     await this.query("DELETE FROM environments WHERE id = $1", [id]);
   }
   /**
@@ -9475,6 +9510,7 @@ var variableSchema = z5.object({
   key: z5.string(),
   value: z5.string(),
   defaultValue: z5.string(),
+  enabled: z5.boolean().optional().default(true),
   share: z5.boolean()
 });
 var authConfigSchema = z5.object({
@@ -9664,7 +9700,8 @@ var environmentRecordSchema = z8.object({
   createdByUserId: z8.string().nullable(),
   updatedByUserId: z8.string().nullable(),
   deletionLocked: z8.boolean(),
-  marker: z8.string().nullable()
+  marker: z8.string().nullable(),
+  parentUuid: z8.string().nullable()
 });
 var snippetScopeSchema = z8.enum(["pre-request", "post-request", "any"]);
 var snippetRecordSchema = z8.object({
@@ -9733,7 +9770,8 @@ var createEnvironmentBodySchema = z8.object({
 var updateEnvironmentBodySchema = z8.object({
   name: z8.string().trim().min(1),
   variables: z8.array(variableSchema),
-  marker: sidebarMarkerBodySchema
+  marker: sidebarMarkerBodySchema,
+  parentUuid: z8.union([z8.string().trim().min(1), z8.null()]).optional()
 });
 var createSnippetBodySchema = z8.object({
   name: z8.string().trim().min(1),
@@ -11776,7 +11814,8 @@ async function registerEnvironmentRoutes(app, db) {
           request.body.name,
           request.body.variables,
           user.id,
-          request.body.marker
+          request.body.marker,
+          request.body.parentUuid
         );
         return reply.send(serializeEnvironment(environment));
       } catch (error) {