otomadb / graphql-api Goto Github PK

雑にやりすぎている

いくらなんでも26桁は長すぎると思うので10桁ぐらいで良いと思う（特にVideo, Tagに関しては）

複雑になりすぎるので，updateVideoInNeo4jでVideoに対してタグなどを一斉に更新するというスタイルにする

遅いが，雑にesbuildとswcで試したがtop-level await周りでムチャクチャになった　

タグの登録時には次が与えられる．

• 主要なタグ名 $N^p$
• 追加の名前 $N_1 ,\dots, N_n \quad (0 \leq n)$
• 親タグへの参照 $P_1 ,\dots, P_m \quad (0 \leq m)$

タグ名前空間 $\mathcal{N} = \{ \braket{n,\emptyset},\braket{n,p},\dots, \}$

このとき， $\mathcal{N}$ の中で次の組が重複してはならない．

• $\langle N^p, P_1 \rangle ,\dots, \langle N^p, P_m \rangle$
• $\langle N_1, P_1 \rangle ,\dots, \langle N_1, P_m \rangle ,\dots, \langle N_n, P_1 \rangle ,\dots, \langle N_n, P_m \rangle$

• ユーザーネームが既に登録されている
• メールアドレスが既に登録されている
• その他不明な例外

フロント側で追加するときに無いと困る

#192 に関連して

input RegisterVideoInput {
  primaryTitle: String!
  extraTitles: [String!]
  tags: [ID!]!
  primaryThumbnail: String!
  niconico: [String!] # 追加
}

URLが突っ込まれる

Mylist.isIncludesVideoみたいな名前に変える（タグの場合に困る）

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Rate-Limited

These updates are currently rate-limited. Click on a checkbox below to force their creation now.

• Update adminer Docker digest to ceca91a
• Update postgres:15 Docker digest to 1e51188
• Update redis:7.2 Docker digest to 1d38c52
• Update dependency @connectrpc/connect to v1.1.4
• Update dependency @connectrpc/connect-node to v1.1.4
• Update dependency lefthook to v1.6.10
• Update dependency @connectrpc/connect to v1.4.0
• Update dependency @connectrpc/connect-node to v1.4.0
• Update dependency @graphql-codegen/client-preset to v4.2.5
• Update dependency @graphql-tools/utils to v10.1.2
• Update dependency auth0 to v4.3.1
• Update dependency eslint to v8.57.0
• Update dependency eslint-plugin-unused-imports to v3.1.0
• Update dependency eslint-plugin-vitest to v0.5.1
• Update dependency graphql-scalars to v1.23.0
• Update dependency graphql-yoga to v5.3.0
• Update dependency node to v20.12.2 (node, @types/node)
• Update dependency nodemon to v3.1.0
• Update dependency pino to v8.20.0
• Update dependency typescript to v5.4.5
• Update prisma monorepo to v5.12.1 (minor) (@prisma/client, prisma)
• Update typescript-eslint to v6.21.0 (minor) (@typescript-eslint/eslint-plugin, @typescript-eslint/parser)
• Update dependency eslint to v9
• Update dependency eslint-plugin-n to v17
• Update dependency eslint-plugin-simple-import-sort to v12
• Update typescript-eslint to v7 (major) (@typescript-eslint/eslint-plugin, @typescript-eslint/parser)
• Lock file maintenance
• 🔐 Create all rate-limited PRs at once 🔐

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• Replace dependency npm-run-all with npm-run-all2 5.0.0
• Update bufbuild (minor) (@bufbuild/buf, @bufbuild/protobuf, @bufbuild/protoc-gen-es)
• Update dependency @babel/core to v7.24.4
• Update dependency @connectrpc/protoc-gen-connect-es to v1.4.0
• Update dependency @parcel/watcher to v2.4.1
• Update dependency meilisearch to ^0.38.0
• Update dependency neo4j-driver to v5.19.0
• Update dependency prettier to v3.2.5
• Update getmeili/meilisearch Docker tag to v1.7
• Update neo4j Docker tag to v5.18
• Click on this checkbox to rebase all open PRs at once

Ignored or Blocked

These are blocked by an existing closed PR and will not be recreated unless you click a checkbox below.

• Update dependency @connectrpc/protoc-gen-connect-es to v1.1.4
• Update dependency vitest to v1
• Update postgres Docker tag to v16

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

Signup時にIconを作成しないのであればnullを入れておいてほしい（空の文字列で持つというのは違う気がするので）

なんでこの名前にしたのか忘れた

忘れるから

命名するときに思いつかなかったからhistoryになってるけどactivityのほうが妥当な気はする

原因：うっかりしていた

対策：気をつける

semitagとゴッチャゴチャになる

URLが長すぎて終わっているため

関連: #28

denoのときの名残でコレになってるけど微妙すぎる気がする

と思うがOptionalになっている

無意味タグの登録をAPIから行えるようにする

将来的にはモデレータ権限を持ったユーザだけが追加できるようにしたいが，一旦それは後で

LIKE句でどうにかしているが全角半角の表記ゆれとかにもきちんと対応していきたい

utils/types.tsのResolverArgsが全然間違っているらしく，第一引数にTParentが来ることになっているが，実際には来ない（第一引数にやってくるのはTArgs）

ちゃんとしたサーバー実装に移す

• graphql.jsでparentが無い問題 (#39 )を解決する巧いテクは多分無い気がする
  • あったとして今後似たような問題発生したときに情報が少ないと思う

タグを付ける際に重複する事のないようクライアント側でチェックするが，ビデオ側にしかそういったものを実装していなかったのでタグの候補の数だけクエリを投げることになって良くなさすぎる

いくつかの操作には編集者権限を必要とするように認証を入れる

分かりづらすぎる

CIで3時間ぐらい引っかかってたときがあった

どうするの………？

メモ

• リレーションは貼れない (ユーザーには貼ってもいいかもしれない)
  • 何かを削除する時に履歴まで消えると恐らく困る
• ………最悪全てを JSONB で持つ？