Merge pull request #165 from takker99:extend-metadata

feat(patch): `update()`の第2引数から、`Page`の全データを取り出せるようにした

Author: takker99

.github/workflows/ci.yml

@@ -9,7 +9,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: denoland/setup-deno@v1
         with:
-          deno-version: "1.36.1"
+          deno-version: "v1.x"
       - name: Check fmt
         run: deno fmt --check
       - name: Run lint

.github/workflows/udd.yml

@@ -12,7 +12,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: denoland/setup-deno@v1
         with:
-          deno-version: "1.36.1"
+          deno-version: "v1.x"
       - name: Update dependencies
         run: >
           deno run --allow-net --allow-read --allow-write=deps/

browser/dom/click.ts

@@ -53,8 +53,8 @@ export const holdDown = async (
     target: element,
     clientX: options.X,
     clientY: options.Y,
-    pageX: options.X + window.scrollX,
-    pageY: options.Y + window.scrollY,
+    pageX: options.X + globalThis.scrollX,
+    pageY: options.Y + globalThis.scrollY,
   });
   const mouseOptions = {
     button: options.button ?? 0,

browser/dom/extractCodeFiles.test.ts

@@ -1,7 +1,7 @@
 import { extractCodeFiles } from "./extractCodeFiles.ts";
 import { Line } from "../../deps/scrapbox.ts";
 import { assertSnapshot } from "../../deps/testing.ts";
-import sample from "./sample-lines1.json" assert { type: "json" };
+import sample from "./sample-lines1.json" with { type: "json" };
 
 Deno.test("extractCodeFiles", async (t) => {
   await assertSnapshot(

browser/dom/isHeightViewable.ts

@@ -4,5 +4,5 @@
 
 export const isHeightViewable = (element: HTMLElement): boolean => {
   const { top, bottom } = element.getBoundingClientRect();
-  return top >= 0 && bottom <= window.innerHeight;
+  return top >= 0 && bottom <= globalThis.innerHeight;
 };

browser/dom/motion.ts

@@ -161,7 +161,7 @@ const getVisibleLineCount = (): number => {
   if (clientHeight === undefined) {
     throw Error("Could not find .line:last-of-type");
   }
-  return Math.round(window.innerHeight / clientHeight);
+  return Math.round(globalThis.innerHeight / clientHeight);
 };
 
 /** 半ページ上にスクロールする

browser/dom/open.ts

@@ -58,14 +58,14 @@ export const open = (
     options?.newTab !== false &&
     (options?.newTab === true || project !== scrapbox.Project.name)
   ) {
-    window.open(url);
+    globalThis.open(url);
     return;
   }
   if (
     options?.reload !== false &&
     (options?.reload === true || project !== scrapbox.Project.name)
   ) {
-    window.open(url, "_self");
+    globalThis.open(url, "_self");
     return;
   }
 

browser/websocket/_codeBlock.ts

@@ -1,6 +1,6 @@
 import { Change, Socket, wrap } from "../../deps/socket.ts";
+import { Page } from "../../deps/scrapbox-rest.ts";
 import { TinyCodeBlock } from "../../rest/getCodeBlocks.ts";
-import { HeadData } from "./pull.ts";
 import { getProjectId, getUserId } from "./id.ts";
 import { pushWithRetry } from "./_fetch.ts";
 
@@ -14,7 +14,7 @@ export interface CodeTitle {
 /** コミットを送信する一連の処理 */
 export const applyCommit = async (
   commits: Change[],
-  head: HeadData,
+  page: Page,
   projectName: string,
   pageTitle: string,
   socket: Socket,
@@ -26,9 +26,9 @@ export const applyCommit = async (
   ]);
   const { request } = wrap(socket);
   return await pushWithRetry(request, commits, {
-    parentId: head.commitId,
+    parentId: page.commitId,
     projectId: projectId,
-    pageId: head.pageId,
+    pageId: page.id,
     userId: userId_,
     project: projectName,
     title: pageTitle,

browser/websocket/deletePage.ts

@@ -20,7 +20,7 @@ export const deletePage = async (
   options?: DeletePageOptions,
 ): Promise<void> => {
   const [
-    { pageId, commitId: parentId, persistent },
+    { id: pageId, commitId: parentId, persistent },
     projectId,
     userId,
   ] = await Promise.all([

browser/websocket/makeChanges.ts

@@ -1,19 +1,18 @@
 import { diffToChanges } from "./diffToChanges.ts";
-import type { Line } from "../../deps/scrapbox.ts";
-import { Block, Node, parse } from "../../deps/scrapbox.ts";
+import { Block, Line, Node, parse } from "../../deps/scrapbox.ts";
+import { Page } from "../../deps/scrapbox-rest.ts";
 import type { Change } from "../../deps/socket.ts";
-import type { HeadData } from "./pull.ts";
 import { toTitleLc } from "../../title.ts";
 import { parseYoutube } from "../../parser/youtube.ts";
 
 export interface Init {
   userId: string;
-  head: HeadData;
+  page: Page;
 }
 export function* makeChanges(
   left: Pick<Line, "text" | "id">[],
   right: string[],
-  { userId, head }: Init,
+  { userId, page }: Init,
 ): Generator<Change, void, unknown> {
   // 改行文字が入るのを防ぐ
   const right_ = right.flatMap((text) => text.split("\n"));
@@ -24,7 +23,7 @@ export function* makeChanges(
 
   // titleの差分を入れる
   // 空ページの場合もタイトル変更commitを入れる
-  if (left[0].text !== right_[0] || !head.persistent) {
+  if (left[0].text !== right_[0] || !page.persistent) {
     yield { title: right_[0] };
   }
 
@@ -38,18 +37,18 @@ export function* makeChanges(
   // リンクと画像の差分を入れる
   const [links, projectLinks, image] = findLinksAndImage(right_.join("\n"));
   if (
-    head.links.length !== links.length ||
-    !head.links.every((link) => links.includes(link))
+    page.links.length !== links.length ||
+    !page.links.every((link) => links.includes(link))
   ) {
     yield { links };
   }
   if (
-    head.projectLinks.length !== projectLinks.length ||
-    !head.projectLinks.every((link) => projectLinks.includes(link))
+    page.projectLinks.length !== projectLinks.length ||
+    !page.projectLinks.every((link) => projectLinks.includes(link))
   ) {
     yield { projectLinks };
   }
-  if (head.image !== image) {
+  if (page.image !== image) {
     yield { image };
   }
 }