scrapbox-userscript-std/PR:108
merge済み

(nit) convert named functions to arrow functions
折角なのでテストを加えたい
getCodeBlocks
filterを挟めるようにしてあるけれど、コードブロックの収集中にフィルター処理を挟んでいるのが良くない気がしたMijinko_SD
最後のreturnでArray.prototype.filter()を使うほうが合理的
updateCodeBlock
...
makeTitleChangeCommit
isCodeFile
これの用途がいまいちわからないtakker
unknown ではなく、いくらか型を絞ってしまってもいいのでは？
それはupdateCodeBlock()のnewCodeの無駄に種類の多い引数の型を判別するために用意したものですMijinko_SD
unknownにしたのは他でも使えるようにするためだけなので、ぶっちゃけ不要です
これによる型判別を使用せずに、updateCodeBlock()の引数を newCode: {content: string | string[]} | CodeFile にするという手もある
これだと関数内で newCode を使うのが楽になる反面、呼び出し方が若干複雑になるというデメリットもある
どちらが良いかわからなかった
flatCodeBodies
まあこれはいいかtakker
よく使う処理を複数箇所に書きたくないので関数化しましたMijinko_SD
なるほどtakker
makeCommits
makeIndent
makeCodeBlockTitle
テストコードの書き方（というかそもそもテストで何をするのか）がよくわからなかったので、多分人任せになるかもしれません…Mijinko_SD
getCodeBlocks() と isCodeFile() と extractFromCodeTitle() の3つの関数のテストコードを書いてみましたMijinko_SD
初めて書いたので、良くない点や間違っている点があるかもしれません
あれば教えていただけると幸いです
了解ですtakker
WebSocketの通信を挟む関数や、exportしていない関数のテストコードの書き方はわからなかったので、まだ書いていません
exportしてないやつは無理ですtakker
通信系はmock使わないと無理ですtakker
scrapbox.io serverの挙動を全部emulateしなきゃいけないのでやりたくない……
fetchだけならscrapbox-userscript-icon-suggestionでいくつか実装例がありますが……やりたくなさすぎて放置している……
あり？っているの？takker
あーなるほど。 isSimpleCodeFile をしているファイル中で定義しているから、 deno.ns を入れる必要があるわけか
isSimpleCodeFile.ts に分離したほうがいいと思う
了解しましたMijinko_SD
getCodeBlocks は /rest でいいと思いますtakker
pull() は getPage() をちょこっとwrapしただけ。そもそも必要性があやしい
組み込み型定義の違いでdirectoryを分けたいので、 /rest に置くのが適切
/rest はどの環境でも使えるファイルを置いている
/websocket はbrowser環境でしか動かせないので、ここに getCodeBlocks を入れると、不要な制約がかかってしまう
ちなみに /dom と /websocket を分けているのは、importの副作用による不要なコードが混入するのを防ぐため
/websocket をimportに含めるとscrapbox-parserのコードが含まれてしまう
これは /dom だけ使うなら不要
了解ですMijinko_SD
PRにtakkerもcommitしたいけどできるのかな？takker
PRにcommitできるのはauthorだけ？
うまいこと複数人が直接commitできるbranchを生やせばいいのかな？
できるならできるようにしたいMijinko_SD
updateCodeFile.ts の CodeFile の適切な名前を考えたいMijinko_SD
browser/dom/extractCodeFiles.ts の CodeFile と名前がぶつかってしまっている
思いつかない……………
SimpleCodeFile へ変更した
自分も混乱してきました
でもあんまり長い名前は入れたくないんですよね
namespace (TypeScript)を導入するしかないかな？
長い名前が嫌というわけではないので大丈夫です（）Mijinko_SD
もちろん、長いより短いに越したことはありませんが
デバッグ文に色（limegreen）をつけた

local_test/fmt-watch.ps1
$files = @(
"./local_test/test.ts"
"./local_test/getCodeBlocks.test.ts"
"./browser/websocket/mod.ts"
"./browser/websocket/updateCodeBlock.ts"
"./browser/websocket/updateCodeFile.ts"
"./browser/websocket/updateCodeFile.test.ts"
"./browser/websocket/isSimpleCodeFile.ts"
"./browser/websocket/isSimpleCodeFile.test.ts"
"./browser/websocket/_codeBlock.ts"
"./browser/websocket/_codeBlock.test.ts"
"./rest/mod.ts"
"./rest/getCodeBlocks.ts"
"./rest/getCodeBlocks.test.ts"
) | ForEach-Object {$PSScriptRoot + "/../" + $PSItem}
deno fmt --watch $files
deno.jsonにいくつかコマンドを定義しようかなtakker
deno task <なんとか> で実行できるようになる

2023-02-26
07:55:14 もうmergeしようかと思っているtakker
mergeせずに放置する理由がない
reviewはまだできてないけど、このままreviewするまでmergeしないとかしていると、永久にmergeされない
定期的に時間をとって少しずつreviewする方法もあるが、それだと自転車置き場レベルの細かい指摘を何度も何度もすることになって、Mijinko_SDさんが大変になる
たぶん動いているだろうし、mergeしちゃっても大丈夫だと思う
一応deno testだけはやっとく
個人的にテストした限りでは多分大丈夫だと思いますMijinko_SD

2023-02-17 14:23:49 必要になったので先に#109をmergeします。すみませんtakker
よきMijinko_SD

getCodeBlocks()でコードブロックを取得する際、タイトル行の行IDで絞り込めるようにしましたMijinko_SD
ついでにフィルター周りのコードを少し簡略化しました
挙動は変えずに内部処理だけ変更した

deno fmtを度々やり忘れてしまっていてすみませんMijinko_SD
プッシュ前に deno fmt を入力するように心がけます

プッシュ前にやることMijinko_SD
$ deno fmt
フォーマットし忘れの阻止
VSCodeに常時Denoでフォーマットする設定を入れたので不要
{
// 中略
"settings": {
"deno.enable": true,
"editor.defaultFormatter": "denoland.vscode-deno",
"editor.formatOnSave": true
}
}
$ deno test --allow-read
テストファイルやテスト対象に異常があればここでエラーが出る
$ wsl deno check --reload ./**/*.ts
Windows環境でワイルドカードを使用する方法がわからなかったので、WSL 2を噛ませる
2023/03/01 --reload を含めて更新するようにしないとpush後にエラーを吐いたMijinko_SD
scrapbox-parser周りかesm.sh関係のエラーだと思われ
pre-commitを使うと自動化できるかも？takker
提案しておいてなんですが、自分も一度も使ったことないです……
名前だけ知ってる
今度試しに使ってみようかな

Related