@drop2run/mcp 0.10.0 → 0.11.0

package/README.md

@@ -88,7 +88,10 @@ lose it.
 | `list_folders` | Lists the account's folders, each with the path `folder` accepts |
 | `delete_site` | Deletes a site permanently, subdomain included — requires the subdomain repeated as `confirm` |
 | `get_site` | Reads a site's settings, whether it is paused, and its versions with the live one marked |
-| `update_site` | Changes a site's name, serving mode, password, forms, scheduled takedown or folder |
+| `update_site` | Changes a site's name, serving mode, password, invite-only, comments, forms, scheduled takedown or folder |
+| `list_people` | Lists who a site is shared with, what each may do, and whether only they can open it |
+| `invite_people` | Invites addresses the person gave — to view, or to view and comment — and emails each one |
+| `remove_person` | Takes one address off a site's list |
 | `pause_site` | Takes a site off the air, keeping its files, versions and subdomain |
 | `resume_site` | Puts a paused site back on the air |
 | `rollback_site` | Serves an earlier version again, by its id from `get_site` |
@@ -117,6 +120,12 @@ default; `expiryAction: "delete"` removes it instead, and asks for the subdomain
 `delete_site` does. Settings your plan does not include are refused with the reason, and `get_site` says
 which ones those are.
 
+To share a site for review, `invite_people` puts the addresses the person gave on its list and emails each
+one; `update_site` with `invitedOnly: true` makes the list the only way in, and `comments` decides who may
+comment — `invited` for the people invited with the comment role, `anyone` for anybody signed in. Both are
+plan features, refused with the reason where the plan lacks them. The tool description tells the model never
+to invite an address the person did not give it.
+
 The comment tools close the loop on a site somebody is reviewing: `list_comments` gives each thread's page
 and the words or element it points at, so the source can be found; after the fix is published,
 `reply_comment` answers on the page. Publish first and reply second, or the commenter opens the old

package/dist/index.js

@@ -177,10 +177,7 @@ async function folderFromListing(credentials, siteId, subdomain) {
 }
 async function getSite(credentials, siteId) {
   const [site, invitedOnly] = await Promise.all([
-    call(
-      credentials,
-      `sites/${encodeURIComponent(siteId)}`
-    ),
+    call(credentials, `sites/${encodeURIComponent(siteId)}`),
     readInvitedOnly(credentials, siteId)
   ]);
   const folderId = site.folderId !== void 0 ? site.folderId : await folderFromListing(credentials, site.siteId, site.subdomain);
@@ -202,6 +199,8 @@ async function getSite(credentials, siteId) {
     expiresAt: site.expiresAt,
     expiryAction: actionOf(site.expiresAt, site.expiryAction ?? ""),
     scheduledExpiryAvailable: site.scheduledExpiryAvailable,
+    comments: audienceOf(site.commentAudience),
+    commentsAvailable: site.feedbackAvailable === true,
     folderId,
     deploys: site.deploys.map((deploy2) => ({
       deployId: deploy2.deployId,
@@ -226,9 +225,11 @@ async function updateSiteSettings(credentials, siteId, change) {
   if (change.expiryAction !== void 0) body.expiryAction = change.expiryAction;
   if (change.formsEnabled !== void 0) body.formsEnabled = change.formsEnabled;
   if (change.folderId !== void 0) body.folderId = change.folderId;
+  if (change.invitedOnly !== void 0) body.invitedOnly = change.invitedOnly;
+  if (change.comments !== void 0) body.commentAudience = API_AUDIENCE[change.comments];
   if (Object.keys(body).length === 0) throw new Error("Nothing to change — name a setting.");
   const wasInvitedOnly = change.password !== void 0 && change.password !== "" ? await readInvitedOnly(credentials, siteId) : null;
-  const settings = await call(credentials, `sites/${encodeURIComponent(siteId)}`, {
+  const { commentAudience, ...settings } = await call(credentials, `sites/${encodeURIComponent(siteId)}`, {
     method: "PATCH",
     body: JSON.stringify(body)
   });
@@ -237,9 +238,19 @@ async function updateSiteSettings(credentials, siteId, change) {
     ...settings,
     expiryAction: actionOf(settings.expiresAt, settings.expiryAction ?? ""),
     invitedOnly,
+    comments: audienceOf(commentAudience),
     replacedInviteOnly: wasInvitedOnly === true && invitedOnly === false
   };
 }
+const API_AUDIENCE = {
+  off: "Off",
+  invited: "Invited",
+  anyone: "Anyone"
+};
+function audienceOf(value) {
+  const lower = value?.toLowerCase();
+  return lower === "off" || lower === "invited" || lower === "anyone" ? lower : null;
+}
 async function setSitePaused(credentials, siteId, action) {
   const state = await call(
     credentials,
@@ -354,6 +365,34 @@ async function setCommentResolved(credentials, siteId, threadId, resolved) {
     { method: "POST" }
   );
 }
+const PEOPLE_PER_REQUEST = 50;
+async function listPeople(credentials, siteId) {
+  const page = await call(credentials, `sites/${encodeURIComponent(siteId)}/viewers?pageSize=${PEOPLE_PER_REQUEST}`);
+  return {
+    invitedOnly: page.invitedOnly,
+    total: page.total,
+    limit: page.limit,
+    people: page.viewers.map((row) => ({
+      email: row.email,
+      role: enumName(row.role),
+      owner: row.owner,
+      status: enumName(row.status)
+    }))
+  };
+}
+async function invitePeople(credentials, siteId, emails, role) {
+  return await call(credentials, `sites/${encodeURIComponent(siteId)}/viewers`, {
+    method: "POST",
+    body: JSON.stringify({ emails, role: role === "comment" ? "Comment" : "View", notify: true })
+  });
+}
+async function removePerson(credentials, siteId, email) {
+  await call(
+    credentials,
+    `sites/${encodeURIComponent(siteId)}/viewers?email=${encodeURIComponent(email)}`,
+    { method: "DELETE" }
+  );
+}
 const VERSIONS_LISTED = 10;
 function servingModeOf$1(site) {
   if (site.spaMode) return "spa";
@@ -382,6 +421,7 @@ function describeSite(site, folderPath) {
     `  mode      ${servingModeOf$1(site)}${site.modeIsManual ? "" : " (detected on each publish)"}`,
     `  password  ${site.passwordProtected ? "on" : "off"}${unavailable(site.passwordProtectionAvailable)}`,
     `  forms     ${site.formsEnabled ? "on" : "off"}${unavailable(site.formsAvailable)}`,
+    `  comments  ${site.comments ?? "unknown"}${unavailable(site.commentsAvailable)}`,
     `  takedown  ${site.expiresAt === null ? "none" : `${site.expiresAt} (${site.expiryAction})`}${unavailable(site.scheduledExpiryAvailable)}`,
     `  folder    ${site.folderId === null ? "(top level)" : folderPath ?? site.folderId}`,
     "",
@@ -816,13 +856,14 @@ const FILE_TYPES = [
     reader: true,
     editable: true
   },
-  // Source the viewer highlights. A link straight to one of these hands over the file.
+  // Source the viewer highlights. Inside a documents site a link straight to one opens it in the
+  // reader; on an ordinary site, and with `?raw`, it hands over the file.
   {
     ext: ".css",
     mime: "text/css; charset=utf-8",
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: true
   },
   {
@@ -830,7 +871,7 @@ const FILE_TYPES = [
     mime: "text/javascript; charset=utf-8",
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: true
   },
   {
@@ -838,7 +879,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -846,7 +887,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -854,7 +895,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -862,19 +903,18 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   // The languages an assistant writes most, beyond the web's own. Same row as `.ts`: shown as source
-  // when somebody opens one in the reader or publishes it on its own, handed over as a file when a
-  // link names it directly, and no media type — adding one changes what a browser does with a
+  // in the reader, handed over as a file with `?raw` or on an ordinary site, and no media type — adding one changes what a browser does with a
   // published file, which I10 makes a decision of its own rather than a side effect of this list.
   {
     ext: ".py",
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -882,7 +922,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -890,7 +930,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -898,7 +938,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -906,7 +946,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -914,7 +954,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -922,7 +962,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -930,7 +970,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -938,7 +978,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -946,7 +986,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -954,7 +994,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -962,7 +1002,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -970,7 +1010,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -978,7 +1018,7 @@ const FILE_TYPES = [
     mime: null,
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: false
   },
   {
@@ -986,7 +1026,7 @@ const FILE_TYPES = [
     mime: "application/xml",
     kind: "code",
     document: false,
-    reader: false,
+    reader: true,
     editable: true
   },
   // Pictures, shown with an img element.
@@ -1057,25 +1097,25 @@ const FILE_TYPES = [
     reader: false,
     editable: false
   },
-  // A customer's own pages. Never rendered inside the reader.
+  // A customer's own pages. Inside a documents site a navigation opens the reader, which frames the
+  // page; `?raw` and every ordinary site serve the page itself.
   {
     ext: ".html",
     mime: "text/html; charset=utf-8",
     kind: "html",
     document: false,
-    reader: false,
+    reader: true,
     editable: true
   },
   {
-    // ⚠️ No media type, and that is the state of the code rather than a decision: `TYPES` in
-    // `headers.ts` has never held `htm`, so a published `.htm` is served as octet-stream and
-    // downloads instead of rendering. Recorded here rather than fixed, because this table was
-    // introduced as a refactor and a refactor that changes what a URL answers is not one.
+    // An HTML page in every respect the Worker decides by extension: media type, the revalidated
+    // cache policy invariant I7 requires, and the badge's `isHtml`. Missing from `TYPES` it used to
+    // download, which inside the reader meant a frame that downloaded instead of showing a page.
     ext: ".htm",
-    mime: null,
+    mime: "text/html; charset=utf-8",
     kind: "html",
     document: false,
-    reader: false,
+    reader: true,
     editable: true
   },
   // Everything below renders as nothing — the viewer offers a download — but still needs a media
@@ -2154,7 +2194,7 @@ async function signInWithCode(waitSeconds, replace = false) {
     "Call this tool again to keep waiting. Nothing has been changed yet."
   ].join("\n");
 }
-const SERVER_VERSION = "0.10.0";
+const SERVER_VERSION = "0.11.0";
 const CREDENTIAL_WRITE = {
   readOnlyHint: false,
   destructiveHint: false,
@@ -2204,6 +2244,12 @@ const ANSWERS_A_COMMENT = {
   idempotentHint: false,
   openWorldHint: true
 };
+const INVITES_PEOPLE = {
+  readOnlyHint: false,
+  destructiveHint: false,
+  idempotentHint: true,
+  openWorldHint: true
+};
 const MARKS_A_THREAD = {
   readOnlyHint: false,
   destructiveHint: false,
@@ -2292,6 +2338,9 @@ const SETTINGS_OUTPUT = {
     "Whether only invited people can open it, or null when that could not be read. Null does not mean public."
   ),
   formsEnabled: z.boolean().describe("Whether it accepts form submissions."),
+  comments: z.enum(["off", "invited", "anyone"]).nullable().describe(
+    "Who may comment on its pages: nobody, the people invited to comment (only while it is invite-only), or anybody signed in. Null when that could not be read."
+  ),
   expiresAt: z.string().nullable().describe("When it is scheduled to come down, or null."),
   expiryAction: z.string().nullable().describe("What the takedown does: `pause` or `delete`, or null when none is scheduled."),
   folderId: z.string().nullable().describe("The folder it is filed in, or null at the top level.")
@@ -2320,10 +2369,12 @@ function structuredSite(site) {
     expiresAt: site.expiresAt,
     expiryAction: site.expiryAction,
     folderId: site.folderId,
+    comments: site.comments,
     available: {
       password: site.passwordProtectionAvailable,
       forms: site.formsAvailable,
-      scheduledTakedown: site.scheduledExpiryAvailable
+      scheduledTakedown: site.scheduledExpiryAvailable,
+      comments: site.commentsAvailable
     },
     versions: site.deploys.map((deploy2) => ({
       deployId: deploy2.deployId,
@@ -2426,6 +2477,12 @@ function createServer() {
         "a password or a date only when the person gave one, and scheduling a deletion asks for",
         "`confirm` the way delete_site does.",
         "",
+        "To share a site for review: invite_people with the addresses the person gave (each is",
+        "emailed), update_site `invitedOnly: true` so only they can open it, and `comments` to let",
+        "them comment — `invited` for the people invited with the comment role, `anyone` for",
+        "anybody signed in. list_people shows the list; remove_person takes someone off it.",
+        "Never invite an address the person did not give you.",
+        "",
         "People viewing a site can leave comments pinned to its pages. list_comments reads them,",
         "with the page and the words each one points at, so the source can be found and changed.",
         'After changing it, publish first and reply second — a reply that says "fixed" before the',
@@ -2597,7 +2654,8 @@ function createServer() {
         available: z.object({
           password: z.boolean(),
           forms: z.boolean(),
-          scheduledTakedown: z.boolean()
+          scheduledTakedown: z.boolean(),
+          comments: z.boolean()
         }).describe("Which settings the account's plan includes."),
         versions: z.array(VERSION_OUTPUT).describe("Its versions, newest first.")
       },
@@ -2636,6 +2694,12 @@ function createServer() {
         folder: z.string().optional().describe(
           "Folder to move the site to — a path such as Clients/Acme or an id from list_folders — or `root` for the top level. Never creates a folder."
         ),
+        invitedOnly: z.boolean().optional().describe(
+          "True: only the people on the site's list (invite_people) can open it, each signed in. It replaces a password. False: anybody with the link can open it again. Change it only when the person asked who should see the site."
+        ),
+        comments: z.enum(["off", "invited", "anyone"]).optional().describe(
+          "Who may comment on the site's pages: `off`; `invited` — the people invited with the comment role, on an invite-only site; or `anyone` signed in. Only on plans that include comments."
+        ),
         confirm: z.string().optional().describe(
           "Required only with `expiryAction: delete`: the site's subdomain, repeated exactly as the person said it. Ask for it rather than filling it in."
         )
@@ -2643,7 +2707,19 @@ function createServer() {
       outputSchema: SETTINGS_OUTPUT,
       annotations: CHANGES_A_SITE
     },
-    ({ site, name, mode, password, forms, expiresAt, expiryAction, folder, confirm }) => withCredentials(async (credentials) => {
+    ({
+      site,
+      name,
+      mode,
+      password,
+      forms,
+      expiresAt,
+      expiryAction,
+      folder,
+      invitedOnly,
+      comments,
+      confirm
+    }) => withCredentials(async (credentials) => {
       const found = await findSite(credentials, site);
       if (expiryAction !== void 0 && expiresAt === void 0) {
         throw new Error("`expiryAction` only applies with `expiresAt` — send the date too.");
@@ -2662,7 +2738,9 @@ function createServer() {
         // `delete` alongside an empty date would still run its owner-only check for nothing.
         expiryAction: expiresAt === "" ? void 0 : expiryAction,
         formsEnabled: forms,
-        folderId: folder === void 0 ? void 0 : await resolveFolderTarget(credentials, folder)
+        folderId: folder === void 0 ? void 0 : await resolveFolderTarget(credentials, folder),
+        invitedOnly,
+        comments
       };
       const settings = await updateSiteSettings(credentials, found.siteId, change);
       const lines = [
@@ -2670,9 +2748,15 @@ function createServer() {
         `  access    ${accessOf(settings)}`,
         `  mode      ${servingModeOf$1(settings)}`,
         `  forms     ${settings.formsEnabled ? "on" : "off"}`,
+        `  comments  ${settings.comments ?? "unknown"}`,
+        // The one combination that reads as on and does nothing: invited commenters on a site anybody
+        // can open, which the API keeps (it is the owner's choice for when the site goes private).
+        ...settings.comments === "invited" && settings.invitedOnly === false ? [
+          "Comments are set to invited people, but the site is not invite-only, so nobody can comment yet. Set `invitedOnly: true`, or `comments: anyone`."
+        ] : [],
         `  takedown  ${settings.expiresAt === null ? "none" : `${settings.expiresAt} (${settings.expiryAction})`}`,
         ...settings.replacedInviteOnly ? [
-          "The password replaced invite-only: the people on its list can no longer get in without it. The list is kept for switching back in the dashboard."
+          "The password replaced invite-only: the people on its list can no longer get in without it. The list is kept; `invitedOnly: true` switches back."
         ] : [],
         ...settings.live ? ["Reaching every edge takes up to about a minute."] : ["Nothing is published yet, so this applies from the first publish."]
       ];
@@ -2683,6 +2767,7 @@ function createServer() {
         passwordProtected: settings.passwordProtected,
         invitedOnly: settings.invitedOnly,
         formsEnabled: settings.formsEnabled,
+        comments: settings.comments,
         expiresAt: settings.expiresAt,
         expiryAction: settings.expiryAction,
         folderId: settings.folderId
@@ -2748,6 +2833,94 @@ ${promoted.url}`,
       );
     })
   );
+  server.registerTool(
+    "list_people",
+    {
+      title: "List who a site is shared with",
+      description: "Lists the people on a site's list — the ones invite_people added — with what each may do (`view` or `comment`) and whether they have opened it yet. Says whether the list decides who can open the site, which it does only while the site is invite-only.",
+      inputSchema: { site: SITE_INPUT },
+      outputSchema: {
+        invitedOnly: z.boolean().describe("Whether only the people on the list can open the site now."),
+        total: z.number().int().describe("How many are on the list."),
+        limit: z.number().int().describe("The most the plan allows on one site's list."),
+        people: z.array(
+          z.object({
+            email: z.string(),
+            role: z.string().describe("`view` or `comment`."),
+            owner: z.boolean().describe("Whether this is the site's owner."),
+            status: z.string().describe("`opened` or `not-opened`.")
+          })
+        )
+      },
+      annotations: READS_ONLY
+    },
+    ({ site }) => withCredentials(async (credentials) => {
+      const found = await findSite(credentials, site);
+      const list = await listPeople(credentials, found.siteId);
+      const rows = list.people.map(
+        (person) => `  ${person.email}  ${person.role}${person.owner ? ", owner" : ""}, ${person.status}`
+      );
+      return report(
+        [
+          `${list.total} of ${list.limit} on ${found.subdomain}'s list. ` + (list.invitedOnly ? "Only they can open it." : "It is not invite-only, so anybody with the link can open it."),
+          ...rows,
+          ...list.total > list.people.length ? [`  … and ${list.total - list.people.length} more, in the dashboard`] : []
+        ].join("\n"),
+        { ...list }
+      );
+    })
+  );
+  server.registerTool(
+    "invite_people",
+    {
+      title: "Invite people to a site",
+      description: "Adds email addresses to a site's list and emails each new one an invitation from the account. Use only addresses the person gave you — never guess, infer or look one up. `comment` lets them comment as well as open it. Inviting does not make the site private; update_site's `invitedOnly: true` does. On plans with private sharing; all or nothing — one bad address, or a list past the plan's limit, adds nobody.",
+      inputSchema: {
+        site: SITE_INPUT,
+        emails: z.array(z.string().min(3)).min(1).max(50).describe("The addresses, exactly as the person gave them."),
+        role: z.enum(["view", "comment"]).optional().describe("`view` (the default) to open the site, `comment` to open it and comment.")
+      },
+      outputSchema: {
+        added: z.array(z.string()).describe("Addresses added and emailed."),
+        alreadyListed: z.array(z.string()).describe("Addresses already on the list, left as they were and not emailed again."),
+        total: z.number().int().describe("How many are on the list now.")
+      },
+      annotations: INVITES_PEOPLE
+    },
+    ({ site, emails, role }) => withCredentials(async (credentials) => {
+      const found = await findSite(credentials, site);
+      const invited = await invitePeople(credentials, found.siteId, emails, role ?? "view");
+      const detail = await getSite(credentials, found.siteId);
+      return report(
+        [
+          invited.added.length === 0 ? `Nobody new on ${found.subdomain}'s list.` : `Invited ${invited.added.join(", ")} to ${found.subdomain} — each was emailed.`,
+          ...invited.alreadyListed.length === 0 ? [] : [`Already on the list, not emailed again: ${invited.alreadyListed.join(", ")}.`],
+          ...detail.invitedOnly === false ? [
+            "The site is not invite-only yet, so the list does not decide who opens it. update_site with `invitedOnly: true` does."
+          ] : []
+        ].join("\n"),
+        { ...invited }
+      );
+    })
+  );
+  server.registerTool(
+    "remove_person",
+    {
+      title: "Remove someone from a site's list",
+      description: "Takes one address off a site's list. On an invite-only site their access ends within a minute; putting them back is another invitation, with another email.",
+      inputSchema: {
+        site: SITE_INPUT,
+        email: z.string().min(3).describe("The address to remove, from list_people.")
+      },
+      outputSchema: { email: z.string().describe("The address removed.") },
+      annotations: CHANGES_A_SITE
+    },
+    ({ site, email }) => withCredentials(async (credentials) => {
+      const found = await findSite(credentials, site);
+      await removePerson(credentials, found.siteId, email);
+      return report(`Removed ${email} from ${found.subdomain}'s list.`, { email });
+    })
+  );
   server.registerTool(
     "list_comments",
     {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@drop2run/mcp",
-	"version": "0.10.0",
+	"version": "0.11.0",
 	"description": "MCP server that publishes what an AI made — a page, a document or a whole site — to Drop2Run and returns a live URL.",
 	"mcpName": "run.dropto/drop2run",
 	"license": "MIT",