@company-semantics/contracts 86.1.0 → 87.0.0

package/src/notifications/__tests__/render-snapshot.test.ts

@@ -150,39 +150,39 @@ add("security.alert", "Unusual login location", {
 
 const CHAT_BASE = {
   sharedByName: "Alex Rivera",
-  visibility: "private",
+  access: "read",
 } as const;
 add("chat.shared", "Default", {
   ...CHAT_BASE,
   chatTitle: "API design discussion",
-  shareUrl: `${APP}/share/abc123`,
+  chatUrl: `${APP}/chat/abc123`,
   previewText:
     "Here are my thoughts on the REST API design for the new authentication endpoints.",
 });
 add("chat.shared", "No preview", {
   ...CHAT_BASE,
   chatTitle: "Quick sync notes",
-  shareUrl: `${APP}/share/xyz789`,
+  chatUrl: `${APP}/chat/xyz789`,
 });
 add("chat.shared", "Long title", {
   ...CHAT_BASE,
-  visibility: "public",
+  access: "collaborate",
   chatTitle:
     "This is a very long chat title that should be truncated to fit the layout properly",
-  shareUrl: `${APP}/share/long456`,
+  chatUrl: `${APP}/chat/long456`,
   previewText: "The assistant response preview text.",
 });
 add("chat.shared", "Long preview", {
   ...CHAT_BASE,
   chatTitle: "Code review feedback",
-  shareUrl: `${APP}/share/preview789`,
+  chatUrl: `${APP}/chat/preview789`,
   previewText:
     "I reviewed the pull request and found several areas that need improvement. The authentication logic needs better error handling, and the database queries could be optimized.",
 });
 add("chat.shared", "Short", {
   ...CHAT_BASE,
   chatTitle: "Hi",
-  shareUrl: `${APP}/share/short123`,
+  chatUrl: `${APP}/chat/short123`,
   previewText: "Hello!",
 });
 

package/src/notifications/kinds/chat-shared.ts

@@ -3,8 +3,8 @@
  *
  * The only kind that reproduces a real conversation: the chat's title is shown
  * as the user's turn and the preview as the assistant's reply, elided into the
- * call to action. `visibility` is carried by the payload but says nothing here —
- * it is a fact about the share, not about what this notification tells you.
+ * call to action. `access` says whether the recipient can take part or only
+ * read (ADR-CONTRACTS-206); the link is the chat itself, never a token.
  */
 
 import type { ChatUnitItem } from "../content";
@@ -15,7 +15,7 @@ import { NOTICE } from "../text";
 export const chatSharedDefinition: NotificationDefinition<"chat.shared"> = {
   kind: "chat.shared",
   compose: (payload, context) => {
-    const { chatTitle, shareUrl, previewText, sharedByName } = payload;
+    const { chatTitle, chatUrl, access, previewText, sharedByName } = payload;
 
     const items: ChatUnitItem[] = [
       { type: "message", role: "user", text: chatTitle, from: sharedByName },
@@ -29,7 +29,7 @@ export const chatSharedDefinition: NotificationDefinition<"chat.shared"> = {
           ]
         : []),
       { type: "continuation" },
-      { type: "callToAction", label: "SEE MORE", href: shareUrl },
+      { type: "callToAction", label: "OPEN CHAT", href: chatUrl },
     ];
 
     return {
@@ -41,7 +41,13 @@ export const chatSharedDefinition: NotificationDefinition<"chat.shared"> = {
         {
           elements: [
             { type: "greeting" },
-            { type: "body", text: "A chat was shared with you." },
+            {
+              type: "body",
+              text:
+                access === "collaborate"
+                  ? "A chat was shared with you. You can take part in it."
+                  : "A chat was shared with you to read.",
+            },
             { type: "chatUnit", items },
             {
               type: "notice",

package/src/notifications/payloads.ts

@@ -108,11 +108,11 @@ export interface NotificationPayloads {
     sharedByName: string;
     /** Title of the chat being shared */
     chatTitle: string;
-    /** Full URL to view the shared chat */
-    shareUrl: string;
-    /** Share visibility mode */
-    visibility: "private" | "public";
-    /** First ~200 chars of chat content for preview. Generated server-side from snapshot. */
+    /** Full URL of the chat itself — the link that opens it in the recipient's app */
+    chatUrl: string;
+    /** What the share granted: Read only or Collaborate */
+    access: "read" | "collaborate";
+    /** First ~200 chars of chat content for preview. Generated server-side. */
     previewText?: string;
   };
   "share.granted": {

package/src/org/__tests__/view-scopes.test.ts

@@ -12,7 +12,6 @@ describe("VIEW_SCOPE_MAP golden snapshot", () => {
       "internal-admin": "internal.view_admin",
       "execution-detail": "org.view_timeline",
       teams: null,
-      chat: null,
       settings: null,
       "my-work": null,
       "user-md": null,
@@ -34,7 +33,6 @@ describe("getViewScope", () => {
   });
 
   it("returns null for public views", () => {
-    expect(getViewScope("chat")).toBeNull();
     expect(getViewScope("settings")).toBeNull();
     expect(getViewScope("upgrade")).toBeNull();
     // teams is membership-derived, not scope-gated (ADR-CONTRACTS-062).

package/src/org/view-scopes.ts

@@ -33,7 +33,6 @@ export const VIEW_SCOPE_MAP = {
   // not a scope. Repointed from the never-enforced `org.view_teams` to null when
   // that scope was retired from the backend catalog (ADR-CONTRACTS-062 / ADR-BE-256).
   teams: null,
-  chat: null,
   settings: null,
   "my-work": null,
   "user-md": null,