Support nested folders in `config/queries/` for query organization
メンテナーはふだん 3 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 72/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- sql, typescript
調査の方向性
まず、plugins/server/utils.js の AppManager.getAppQuery()、getQueries() と、typegen の generateQueriesFromDescribe() を読み、続いてそれらの flat readdir と path.basename の扱いを調べます。ネストされた .sql ファイルが安定した相対 query key に解決され、validation が選択した separator を受け入れ、type generation と frontend lookup がドキュメント化された例をカバーすれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Support nested folders in config/queries/ for query organization
Problem
As the number of SQL queries in a project grows, config/queries/ becomes a flat bag of files with no way to organize them by feature or domain. The only option is naming conventions (e.g. nego_chart_data.sql, trip_summary.sql), which doesn't scale well and provides no structural separation.
Current behavior
AppManagerhardcodesqueriesDirtopath.resolve(process.cwd(), "config/queries")fs.readdir()is non-recursive — subdirectories are silently ignored- Query key validation (
/^[a-zA-Z0-9_-]+$/) rejects any delimiter that could encode a path generate-types(typegen) also uses flatreaddir+path.basename
Proposed behavior
Allow queries to live in subdirectories:
config/queries/
negotiations/
chart_data.sql
spend_summary.sql
dashboard/
trip_summary.sql
trips_by_day.sql
docs/
annotations.obo.sql
Query key derivation would use the relative path with a separator (e.g. / or .):
negotiations/chart_dataornegotiations.chart_data
Frontend usage:
useAnalyticsQuery('negotiations/chart_data', params)
Minimal change required
The fix is small — roughly 3-4 locations need updating:
AppManager.getAppQuery()— changefs.readdir()tofs.readdir(dir, { recursive: true })and adjust filename matching to include relative subdirectory pathsgetQueries()inplugins/server/utils.js— same recursive readdir changegenerateQueriesFromDescribe()in typegen — derive query names from relative paths instead of justpath.basename()- Query key validation regex — allow
/or.as a path separator
Workarounds
Currently using prefix naming (nego_*, trip_*) which works but doesn't provide IDE folder navigation or structural grouping benefits.
- 主要言語
- TypeScript
- スター
- 119
- フォーク
- 36
- 平均マージ
- 4日 16時間
- マージ済み PR（30日）
- 37
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
databricks/appkit のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
databricks/appkit#729 ·
メンテナーはふだん 3 日以内に返信
-
@databricks/lakebase: pool drops maxLifetimeSeconds and other pg options, so connections never recycle and Unity Catalog claims access fails after 4 hours対応中かも @billmcmillin が 5 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 20/100
databricks/appkit#718 ·
メンテナーはふだん 3 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
databricks/appkit#699 ·
メンテナーはふだん 3 日以内に返信
-
難易度 3/5 半日 初心者へのやさしさ 42/100
databricks/appkit#656 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 28/100
databricks/appkit#648 · コメント 2 件 ·
メンテナーはふだん 3 日以内に返信
databricks/appkit の issue をすべて見る
似ている issue
-
Add "append" and "prepend" slots for vue-vuetify extended renderers対応中かも @simeyc が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
eclipsesource/jsonforms#2668 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
README の通知条件を実際の判定処理に合わせる対応中かも @unhappychoice が今日担当しました。 オープンfactory:in-progress priority:medium
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
unhappychoice/oss-issue-opener#395 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug Website
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
OWASP/OWASP-Hunting#88 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信