@basaltkit/comments 1.0.2 → 1.1.0

package/dist/comments.d.ts

@@ -12,6 +12,12 @@ export declare class CommentTenantRequiredError extends BasaltError {
 export interface CommentNode extends Comment {
     replies: CommentNode[];
 }
+/**
+ * Store key every comment is filed under when the app has no tenancy at all.
+ * The {@link CommentStore} contract is tenant-keyed, so a single-tenant app
+ * still needs one stable key — it just shouldn't have to invent it.
+ */
+export declare const SINGLE_TENANT_SCOPE = "default";
 export interface CommentsOptions {
     store?: CommentStore;
     hooks?: HookBus;
@@ -36,11 +42,25 @@ export interface ResourceComments {
  * updates (@basaltkit/realtime) and notifications wire up without coupling.
  */
 export declare class Comments {
+    /**
+     * Whether the host app registered `@basaltkit/tenancy`. `commentsPlugin`
+     * wires this to the container's `'tenancy:active'` metadata marker — a
+     * signal, not an import, so this generic package never depends on the
+     * opt-in SaaS layer. Defaults to `false` (single-tenant).
+     */
+    private readonly tenancyActive;
     private readonly store;
     private readonly hooks;
     private readonly mentionPattern;
     private readonly now;
-    constructor(options?: CommentsOptions);
+    constructor(options?: CommentsOptions, 
+    /**
+     * Whether the host app registered `@basaltkit/tenancy`. `commentsPlugin`
+     * wires this to the container's `'tenancy:active'` metadata marker — a
+     * signal, not an import, so this generic package never depends on the
+     * opt-in SaaS layer. Defaults to `false` (single-tenant).
+     */
+    tenancyActive?: () => boolean);
     on(resourceType: string, resourceId: string, tenantId?: string): ResourceComments;
     get(id: string, tenantId?: string): Promise<Comment | null>;
     edit(id: string, body: string, tenantId?: string): Promise<Comment>;
@@ -50,5 +70,12 @@ export declare class Comments {
     private add;
     private mutate;
     private mentions;
+    /**
+     * The tenant a call is scoped to.
+     *
+     * With `@basaltkit/tenancy` registered an unresolvable tenant is an error: an
+     * unscoped read or write would cross tenants. Without it there is no tenant
+     * dimension, so every comment shares {@link SINGLE_TENANT_SCOPE}.
+     */
     private tenant;
 }

package/dist/comments.js

@@ -13,6 +13,12 @@ export class CommentTenantRequiredError extends BasaltError {
         super('COMMENT_TENANT_REQUIRED', 'A tenant is required — pass tenantId or run inside a tenant context.');
     }
 }
+/**
+ * Store key every comment is filed under when the app has no tenancy at all.
+ * The {@link CommentStore} contract is tenant-keyed, so a single-tenant app
+ * still needs one stable key — it just shouldn't have to invent it.
+ */
+export const SINGLE_TENANT_SCOPE = 'default';
 const buildTree = (comments) => {
     const nodes = new Map(comments.map((c) => [c.id, { ...c, replies: [] }]));
     const roots = [];
@@ -31,11 +37,20 @@ const buildTree = (comments) => {
  * updates (@basaltkit/realtime) and notifications wire up without coupling.
  */
 export class Comments {
+    tenancyActive;
     store;
     hooks;
     mentionPattern;
     now;
-    constructor(options = {}) {
+    constructor(options = {}, 
+    /**
+     * Whether the host app registered `@basaltkit/tenancy`. `commentsPlugin`
+     * wires this to the container's `'tenancy:active'` metadata marker — a
+     * signal, not an import, so this generic package never depends on the
+     * opt-in SaaS layer. Defaults to `false` (single-tenant).
+     */
+    tenancyActive = () => false) {
+        this.tenancyActive = tenancyActive;
         this.store = options.store ?? new MemoryCommentStore();
         this.hooks = options.hooks;
         this.mentionPattern = options.mentionPattern ?? /@([\w-]+)/g;
@@ -115,10 +130,19 @@ export class Comments {
                 ids.add(match[1]);
         return [...ids];
     }
+    /**
+     * The tenant a call is scoped to.
+     *
+     * With `@basaltkit/tenancy` registered an unresolvable tenant is an error: an
+     * unscoped read or write would cross tenants. Without it there is no tenant
+     * dimension, so every comment shares {@link SINGLE_TENANT_SCOPE}.
+     */
     tenant(explicit) {
         const id = explicit ?? tryCtx()?.['tenant']?.id;
-        if (!id)
+        if (id)
+            return id;
+        if (this.tenancyActive())
             throw new CommentTenantRequiredError();
-        return id;
+        return SINGLE_TENANT_SCOPE;
     }
 }

package/dist/index.d.ts

@@ -1,3 +1,3 @@
-export { Comments, CommentNotFoundError, CommentTenantRequiredError, type CommentsOptions, type AddCommentInput, type ResourceComments, type CommentNode, } from './comments.js';
+export { Comments, CommentNotFoundError, CommentTenantRequiredError, SINGLE_TENANT_SCOPE, type CommentsOptions, type AddCommentInput, type ResourceComments, type CommentNode, } from './comments.js';
 export { MemoryCommentStore, type Comment, type CommentStore, type CommentPatch } from './store.js';
 export { commentsPlugin, commentRoutes, COMMENTS, type CommentsPluginOptions } from './plugin.js';

package/dist/index.js

@@ -1,3 +1,3 @@
-export { Comments, CommentNotFoundError, CommentTenantRequiredError, } from './comments.js';
+export { Comments, CommentNotFoundError, CommentTenantRequiredError, SINGLE_TENANT_SCOPE, } from './comments.js';
 export { MemoryCommentStore } from './store.js';
 export { commentsPlugin, commentRoutes, COMMENTS } from './plugin.js';

package/dist/plugin.js

@@ -1,4 +1,4 @@
-import { createToken, ctx, definePlugin, BasaltError } from '@basaltkit/core';
+import { createToken, ctx, definePlugin, BasaltError, ensureMetadata } from '@basaltkit/core';
 import { route } from '@basaltkit/http';
 import { z } from 'zod';
 import { Comments } from './comments.js';
@@ -7,7 +7,10 @@ export function commentsPlugin(options = {}) {
     return definePlugin({
         name: 'basalt:comments',
         register({ container, hooks }) {
-            container.singleton(COMMENTS, () => new Comments({ ...options, hooks }));
+            // 'tenancy:active' is tenancyPlugin's marker: how a generic package
+            // learns the app is multi-tenant without importing @basaltkit/tenancy.
+            const metadata = ensureMetadata(container);
+            container.singleton(COMMENTS, () => new Comments({ ...options, hooks }, () => metadata.get('tenancy:active').length > 0));
         },
     });
 }

package/package.json

@@ -1,9 +1,13 @@
 {
   "name": "@basaltkit/comments",
-  "version": "1.0.2",
+  "version": "1.1.0",
+  "engines": {
+    "node": ">=22.5.0"
+  },
   "description": "Comments for Basalt: per-resource threads with @mentions and resolve/reopen, tenant-scoped, emitting events that bridge to realtime and notifications.",
   "license": "MIT",
   "type": "module",
+  "sideEffects": false,
   "exports": {
     ".": {
       "types": "./dist/index.d.ts",
@@ -14,8 +18,8 @@
     "dist"
   ],
   "dependencies": {
-    "@basaltkit/core": "^1.1.2",
-    "@basaltkit/http": "^1.9.1"
+    "@basaltkit/core": "^1.3.1",
+    "@basaltkit/http": "^1.14.0"
   },
   "peerDependencies": {
     "zod": "^3.24.0 || ^4.0.0"
@@ -25,9 +29,9 @@
     "typescript": "^7.0.2",
     "vitest": "^4.1.11",
     "zod": "^3.24.0 || ^4.0.0",
-    "@basaltkit/auth": "^1.6.3",
-    "@basaltkit/tenancy": "^1.3.3",
-    "@basaltkit/fastify": "^1.6.1",
+    "@basaltkit/auth": "^1.8.0",
+    "@basaltkit/fastify": "^1.8.1",
+    "@basaltkit/tenancy": "^1.4.2",
     "@basaltkit/tsconfig": "^0.24.0"
   },
   "publishConfig": {