magic-builder 1.1.0 → 1.2.0

package/README.md

@@ -129,9 +129,9 @@ magic-builder link create --fid <faas-id>
 Submit Markdown to a People performance draft (`perf` is an alias):
 
 ```bash
-magic-builder performance --cookie ./cookie.txt --markdown ./performance.md
-magic-builder perf --cookie "$PEOPLE_COOKIE" --markdown "# Performance summary"
-magic-builder perf --cookie ./cookie.txt --markdown https://example.com/performance.md --dry-run
+magic-builder performance --review-url <people-review-url> --cookie ./cookie.txt --markdown ./performance.md
+magic-builder perf --review-url <people-review-url> --cookie "$PEOPLE_COOKIE" --markdown "# Performance summary"
+magic-builder perf --review-url <people-review-url> --cookie ./cookie.txt --markdown https://example.com/performance.md --dry-run
 ```
 
 Cookie input accepts a raw `Cookie` header, a local path, or a Netscape cookie file. Markdown input accepts literal content, a local path, an HTTP(S) URL, or stdin. The command extracts `x-f-csrf` from the Cookie and does not print the Cookie in its output.
@@ -140,9 +140,11 @@ When `--cookie` and `--cookie-file` are omitted, `performance` automatically rea
 
 ```bash
 magic-builder extract-cookie --curl-file ./request.curl
-magic-builder performance --markdown ./performance.md
+magic-builder performance --review-url <people-review-url> --markdown ./performance.md
 ```
 
+With `--review-url`, the CLI loads the operator and tenant from `/perf/api/user/settings`, then reuses the current draft version, template, units, fields, source ids, and root review id from `/perf/api/foundation/draft`. Business ids are not hard-coded into the generated request.
+
 Extract Cookie from a copied curl command. This is a top-level general-purpose command. The default output is `./cookie.txt`; the file is created with mode `0600`:
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magic-builder",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "CLI for Magic Builder — publish pages, functions, files, and performance drafts",
   "bin": {
     "magic-builder": "bin/magic-builder.js",

package/src/commands/performance.js

@@ -6,17 +6,8 @@ const { success, fail } = require('../lib/output');
 
 const DEFAULTS = {
   endpoint: 'https://people.bytedance.net/perf/api/foundation/v2/draft',
-  reviewId: '7654473253011852917',
-  formId: '7654466592117358592',
-  templateId: '7654466592117358609',
-  unitId: '7654466592117358775',
-  fieldId: '7654466592117358799',
-  textFieldId: '7654466592117358824',
-  operatorId: '6687792427639817740',
-  sourceId: '7784085065561419062',
-  fieldSourceId: '7662587009851985528',
-  rootReviewId: '7657808756893306935',
-  version: 5,
+  settingsEndpoint: 'https://people.bytedance.net/perf/api/user/settings',
+  draftEndpoint: 'https://people.bytedance.net/perf/api/foundation/draft',
 };
 
 async function run(args, opts) {
@@ -31,8 +22,10 @@ async function run(args, opts) {
   const csrf = getCookie(cookie, 'x-f-csrf');
   if (!csrf) fail('Cookie does not contain x-f-csrf.', 'E_INVALID_ARGS');
 
-  const config = resolveConfig(opts);
-  const payload = buildPayload(markdown, config);
+  const config = await resolveConfig(opts, cookie);
+  const payload = config.draft
+    ? buildPayloadFromDraft(markdown, config.draft, config)
+    : buildPayload(markdown, config);
   if (opts.dryRun) {
     success({ dryRun: true, endpoint: config.endpoint, payload }, opts);
     return;
@@ -112,30 +105,82 @@ function getCookie(cookie, name) {
   return '';
 }
 
-function resolveConfig(opts) {
-  const reviewId = String(opts.reviewId || DEFAULTS.reviewId);
-  const formId = String(opts.formId || DEFAULTS.formId);
-  const templateId = String(opts.templateId || DEFAULTS.templateId);
-  const tenantId = String(opts.tenantId || getCookie(String(opts.cookie || ''), 'tenant_id') || '6685321562717324807');
+async function resolveConfig(opts, cookie) {
+  const review = parseReviewLocation(opts.reviewUrl || opts.referer, opts.reviewId, opts.formId);
+  if (review.reviewId && review.formId) {
+    const settings = await requestJson(opts.settingsEndpoint || DEFAULTS.settingsEndpoint, cookie, review.referer);
+    const operatorId = String(opts.operatorId || settings?.data?.user?.id || '');
+    const tenantId = String(opts.tenantId || settings?.data?.tenant?.id || '');
+    if (!operatorId) fail('Unable to resolve operator id from /perf/api/user/settings.', 'E_REQUEST_FAILED');
+    const key = `${review.reviewId}__confirm_invitation__${operatorId}__${review.formId}`;
+    const draftUrl = new URL(opts.draftEndpoint || DEFAULTS.draftEndpoint);
+    draftUrl.searchParams.set('key', key);
+    const draftResponse = await requestJson(draftUrl.toString(), cookie, review.referer);
+    if (!draftResponse?.success || !draftResponse?.data) fail('Unable to load the current performance draft.', 'E_REQUEST_FAILED');
+    return {
+      endpoint: String(opts.endpoint || DEFAULTS.endpoint),
+      reviewId: review.reviewId,
+      formId: review.formId,
+      operatorId,
+      tenantId,
+      referer: review.referer,
+      uid: String(opts.uid || `magic-builder-${Date.now()}`),
+      version: opts.draftVersion === undefined ? undefined : Number(opts.draftVersion),
+      draft: draftResponse.data,
+    };
+  }
+
+  const required = ['templateId', 'unitId', 'fieldId', 'textFieldId', 'operatorId', 'sourceId', 'fieldSourceId', 'rootReviewId'];
+  const missing = required.filter(name => !opts[name]);
+  if (!opts.reviewId || !opts.formId || missing.length) {
+    fail('Pass --review-url <People review URL> to resolve the current draft automatically.', 'E_INVALID_ARGS');
+  }
   return {
     endpoint: String(opts.endpoint || DEFAULTS.endpoint),
-    reviewId,
-    formId,
-    templateId,
-    unitId: String(opts.unitId || DEFAULTS.unitId),
-    fieldId: String(opts.fieldId || DEFAULTS.fieldId),
-    textFieldId: String(opts.textFieldId || DEFAULTS.textFieldId),
-    operatorId: String(opts.operatorId || DEFAULTS.operatorId),
-    sourceId: String(opts.sourceId || DEFAULTS.sourceId),
-    fieldSourceId: String(opts.fieldSourceId || DEFAULTS.fieldSourceId),
-    rootReviewId: String(opts.rootReviewId || DEFAULTS.rootReviewId),
-    version: Number(opts.version || DEFAULTS.version),
-    tenantId,
-    referer: String(opts.referer || `https://people.bytedance.net/performance/perf/review/${reviewId}/${formId}?mode=editable`),
+    reviewId: String(opts.reviewId),
+    formId: String(opts.formId),
+    templateId: String(opts.templateId),
+    unitId: String(opts.unitId),
+    fieldId: String(opts.fieldId),
+    textFieldId: String(opts.textFieldId),
+    operatorId: String(opts.operatorId),
+    sourceId: String(opts.sourceId),
+    fieldSourceId: String(opts.fieldSourceId),
+    rootReviewId: String(opts.rootReviewId),
+    version: Number(opts.draftVersion || opts.version),
+    tenantId: String(opts.tenantId || ''),
+    referer: String(opts.referer || `https://people.bytedance.net/performance/perf/review/${opts.reviewId}/${opts.formId}?mode=editable`),
     uid: String(opts.uid || `magic-builder-${Date.now()}`),
   };
 }
 
+function parseReviewLocation(value, reviewId, formId) {
+  if (reviewId && formId) {
+    return {
+      reviewId: String(reviewId),
+      formId: String(formId),
+      referer: `https://people.bytedance.net/performance/perf/review/${reviewId}/${formId}?mode=editable`,
+    };
+  }
+  const raw = String(value || '').trim();
+  if (!raw) return { reviewId: '', formId: '', referer: '' };
+  const url = new URL(raw);
+  if (url.origin !== 'https://people.bytedance.net') fail('Review URL must use https://people.bytedance.net.', 'E_INVALID_ARGS');
+  const match = url.pathname.match(/\/performance\/perf\/review\/(\d+)\/(\d+)/);
+  if (!match) fail('Unable to parse reviewId and formId from --review-url.', 'E_INVALID_ARGS');
+  return { reviewId: match[1], formId: match[2], referer: url.toString() };
+}
+
+async function requestJson(url, cookie, referer) {
+  const response = await fetch(url, {
+    headers: { accept: 'application/json, text/plain, */*', cookie, referer },
+  });
+  const text = await response.text();
+  const data = parseJson(text);
+  if (!response.ok || data === null) throw new Error(`GET ${new URL(url).pathname} failed: HTTP ${response.status}`);
+  return data;
+}
+
 function buildPayload(markdown, config) {
   const now = Date.now();
   const sections = splitMarkdownSections(markdown);
@@ -191,6 +236,34 @@ function buildPayload(markdown, config) {
   };
 }
 
+function buildPayloadFromDraft(markdown, draft, config) {
+  const snapshot = structuredClone(draft);
+  const data = snapshot.data;
+  const keyWorks = data?.keyWorks;
+  if (!keyWorks || !Array.isArray(keyWorks.units)) fail('Current draft does not contain keyWorks.units.', 'E_REQUEST_FAILED');
+  const field = keyWorks.units.flatMap(unit => unit.fields || []).find(item => item.entityType === 'multiple_texts');
+  if (!field) fail('Current draft does not contain a multiple_texts field.', 'E_REQUEST_FAILED');
+  const firstText = field.sub_units?.flat()?.find(item => item?.id);
+  if (!firstText?.id) fail('Current draft does not contain a writable text sub-unit.', 'E_REQUEST_FAILED');
+  const now = Date.now();
+  field.sub_units = createSubUnits(markdown, firstText.id, firstText.source_id || keyWorks.valueSetting?.[0]?.source_id, now);
+  field.updated_time = now;
+  if (config.uid) data.uid = config.uid;
+  return {
+    key: snapshot.key,
+    data,
+    version: config.version ?? Number(snapshot.version),
+  };
+}
+
+function createSubUnits(markdown, textFieldId, sourceId, now = Date.now()) {
+  return splitMarkdownSections(markdown).map((section, index) => {
+    const value = { '0': { ops: markdownToDelta(section), zoneId: '0', zoneType: 'Z' } };
+    if (index > 0) return [{ key: `${index - 1}-${now + index}-${Math.random()}`, id: textFieldId, entityType: 'text', value }];
+    return [{ id: textFieldId, source_id: sourceId, value, json_value: {}, created_time: now, updated_time: now, entityType: 'text' }];
+  });
+}
+
 function splitMarkdownSections(markdown) {
   const text = String(markdown || '').replace(/\r\n?/g, '\n').trim();
   if (!text) return [''];
@@ -261,4 +334,4 @@ function parseJson(text) {
   try { return text ? JSON.parse(text) : {}; } catch (_) { return null; }
 }
 
-module.exports = { run, readCookie, normalizeCookie, getCookie, splitMarkdownSections, markdownToDelta, buildPayload };
+module.exports = { run, readCookie, normalizeCookie, getCookie, parseReviewLocation, splitMarkdownSections, markdownToDelta, buildPayload, buildPayloadFromDraft };

package/src/lib/help.js

@@ -107,8 +107,8 @@ COMMANDS:
   feedback create --feedback <text> [--title <title>] [--summary <text>]
   feedback create --feedback-file <file> [--dry-run]
 
-  performance --cookie <content|file> --markdown <content|file|url> [--dry-run]
-  perf --cookie-file <file> --markdown-file <file> [--dry-run]
+  performance --review-url <url> --cookie <content|file> --markdown <content|file|url> [--dry-run]
+  perf --review-url <url> --cookie-file <file> --markdown-file <file> [--dry-run]
 
   extract-cookie --curl <content|file> [--out <file>|--out-dir <dir>]
 
@@ -197,14 +197,20 @@ SYNTAX:
   performance: `@HELP magic-builder/performance
 
 SYNTAX:
-  magic-builder performance --cookie <content|file> --markdown <content|file|url>
-  magic-builder perf --cookie-file <file> --markdown-file <file>
+  magic-builder performance --review-url <url> --cookie <content|file> --markdown <content|file|url>
+  magic-builder perf --review-url <url> --cookie-file <file> --markdown-file <file>
 
 COOKIE:
   When --cookie and --cookie-file are omitted, ./cookie.txt is used.
 
+DRAFT:
+  --review-url resolves operator, tenant, form ids, and the current version
+  from /perf/api/user/settings and /perf/api/foundation/draft.
+
 OPTIONS:
   --dry-run                    Print the generated payload without sending it
+  --review-url <url>           People performance review URL (recommended)
+  --draft-version <number>     Override the version loaded from the current draft
   --review-id <id>             Override the performance review id
   --form-id <id>               Override the invitation/form id used by the draft key
   --template-id <id>           Override the review template id