Skip to content

system: master data を全件取得へ変更#41

Merged
buty4649 merged 2 commits intomainfrom
system-master-data-full-fetch
Apr 20, 2026
Merged

system: master data を全件取得へ変更#41
buty4649 merged 2 commits intomainfrom
system-master-data-full-fetch

Conversation

@buty4649
Copy link
Copy Markdown
Contributor

Summary

  • xp system master data から --rows / --offset フラグを撤廃し、rows=1000offset を自動インクリメントしてマスタデータを全件取得するように変更
  • JSON 取得時は master.total_count を終端判定に使用し、マージ済みのレスポンスを返す
  • CSV 取得時は 2 ページ目以降を title=false で取得してヘッダを含めず連結、空レスポンスで終了
  • CSV は X-point 側のヘッダ名(日本語名)とフォーマット互換を保つため、従来通り API 側で CSV を生成させる
  • 既定出力を stdout に統一(TTY 検出によるファイル保存を廃止)

Test plan

  • go build ./... / go vet ./... / go test ./... が通ることを確認
  • ユーザ固有マスタ (--type 1) を JSON 取得、master.total_countmaster.data の件数が一致する
  • ユーザ固有マスタ (--type 1) を CSV 取得、ヘッダ 1 行 + データ行の形式で出力される
  • CSV 出力が複数ページに跨がっても、ヘッダが重複しないこと
  • --output 未指定で stdout に出力されること、--output FILE 指定で保存されること

🤖 Generated with Claude Code

--rows / --offset フラグを撤廃し、rows=1000 で offset を自動インクリメン
トしてマスタデータを全件取得するように変更。

- JSON: master.total_count を終端判定に使用し、マージ済みレスポンスを出力
- CSV: 2 ページ目以降は title=false で取得してヘッダを含めず連結、空レス
  ポンスで終了
- 既定出力を stdout に統一

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@buty4649 buty4649 self-assigned this Apr 20, 2026
数値指定 (0/1) は意味がすぐに分からないため、simple (簡易マスタ) /
user (ユーザ固有マスタ) の文字列値を受け取るように変更。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@buty4649 buty4649 enabled auto-merge (squash) April 20, 2026 08:26
@buty4649 buty4649 merged commit 4dec649 into main Apr 20, 2026
2 checks passed
@buty4649 buty4649 deleted the system-master-data-full-fetch branch April 20, 2026 08:26
@pepabo-pr-maker pepabo-pr-maker bot mentioned this pull request Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant