✨ [Trader 15] 日替わり商人の作成 & 商人の日替わり更新システムの実装 by Lapis-LJA · Pull Request #2169 · ProjectTSB/TheSkyBlessing

Lapis-LJA · 2025-11-12T16:31:06Z

Fix #2164

…B/TheSkyBlessing into dev/lapis_daily_trader

Lapis-LJA · 2025-11-24T20:10:38Z

発覚した問題:
島攻略による取引の更新で、取引数がリセットされて何度も同じ神器を購入できてしまう

Copilot

Pull request overview

日替わりで取引内容（交換できる神器）が変化する「日替わり商人（Trader 15）」を追加し、商人の取引更新処理を「攻略度更新」と「日替わり更新」に分離して実装するPRです。

Changes:

Trader 15（日替わり商人）の登録処理と、攻略度に応じた神器プールからの抽選ロジックを追加
商人の取引更新を by_progress（攻略度/バージョン反映）と by_daily（日替わり更新）に分割し、tick から呼び分け
神器レジストリ取得API（通常/色別/隠し枠）と、日替わり更新判定用スコアボードを追加

Reviewed changes

Copilot reviewed 31 out of 31 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
TheSkyBlessing/data/core/functions/load_once.mcfunction	日替わり更新判定用のスコアボードを追加
TheSkyBlessing/data/asset/functions/trader/15/select_artifact/up_to_progress/set_pool.m.mcfunction	攻略度到達までのランクをプールに追加する処理を追加
TheSkyBlessing/data/asset/functions/trader/15/select_artifact/up_to_progress/.mcfunction	up_to_progress 用の抽選フローを追加（隠しプール含む）
TheSkyBlessing/data/asset/functions/trader/15/select_artifact/picks.mcfunction	多次元配列プールから1要素を抽選する共通処理を追加
TheSkyBlessing/data/asset/functions/trader/15/select_artifact/current_progress/set_pool.m.mcfunction	current_progress 用のプール決定（隠し枠への差し替え含む）を追加
TheSkyBlessing/data/asset/functions/trader/15/select_artifact/current_progress/.mcfunction	current_progress 用の抽選フローを追加
TheSkyBlessing/data/asset/functions/trader/15/select_artifact/.mcfunction	攻略度取得〜通常/隠しレジストリ取得〜2種抽選の統合処理を追加
TheSkyBlessing/data/asset/functions/trader/15/register.mcfunction	日替わり商人（ID 15）の登録と取引定義（神器2枠）を追加
TheSkyBlessing/data/asset/functions/trader/15/_index.d.mcfunction	Trader 15 用スコアホルダ宣言を追加
TheSkyBlessing/data/asset_manager/functions/trader/tick/4_interval.mcfunction	tickで日替わり更新を追加し、攻略度更新呼び出し先を変更
TheSkyBlessing/data/asset_manager/functions/trader/register/reset.mcfunction	DailyUpdate フラグをリセット対象に追加
TheSkyBlessing/data/asset_manager/functions/trader/register/construct/set_data.mcfunction	DailyUpdate 用タグ付与と初期更新処理呼び出し先を変更
TheSkyBlessing/data/asset_manager/functions/trader/daily_update.mcfunction	日替わり更新の判定（daytime/gametime）と更新実行を追加
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/update.mcfunction	取引の実更新処理を共通関数として切り出し
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/get_trades.mcfunction	OhMyDat/登録関数から取引一覧を取得する共通処理を追加
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/by_progress/keep_sells/keep.mcfunction	日替わり商人向けに商品維持しつつ更新する処理を追加
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/by_progress/keep_sells/.mcfunction	keep_sells 実行と OhMyDat 反映フローを追加
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/by_progress/check_version.m.mcfunction	バージョン不一致時のみ取引定義を再取得する処理を追加
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/by_progress/.mcfunction	攻略度/バージョン反映の更新フローを新設（DailyUpdate の分岐含む）
TheSkyBlessing/data/asset_manager/functions/trader/common/update_recipe/by_daily.mcfunction	日替わり更新用の更新フローを新設
TheSkyBlessing/data/asset_manager/functions/trader/common/trades_map/.mcfunction	`@within` 注釈の参照先を更新
TheSkyBlessing/data/asset_manager/functions/trader/common/refresh_trades/load_register.m.mcfunction	`@within` 注釈の参照先を更新
TheSkyBlessing/data/asset_manager/functions/trader/common/refresh_trades/.mcfunction	refresh_trades の入口関数へ整理し、取引/Version の更新に特化
TheSkyBlessing/data/asset_manager/functions/trader/_index.d.mcfunction	DailyUpdate タグ宣言と `@within` 参照更新
TheSkyBlessing/data/asset_manager/functions/artifact/load.mcfunction	隠し枠レジストリ用ストレージを追加
TheSkyBlessing/data/asset_manager/functions/artifact/_index.d.mcfunction	get_registry API を `@within` 対象に追加
TheSkyBlessing/data/api/functions/artifact/get_registry/red.mcfunction	レッドシャードの神器レジストリ取得APIを追加
TheSkyBlessing/data/api/functions/artifact/get_registry/normal.mcfunction	通常シャードの神器レジストリ取得APIを追加
TheSkyBlessing/data/api/functions/artifact/get_registry/hide.mcfunction	隠し枠神器のレジストリ取得APIを追加
TheSkyBlessing/data/api/functions/artifact/get_registry/green.mcfunction	グリーンシャードの神器レジストリ取得APIを追加
TheSkyBlessing/data/api/functions/artifact/get_registry/blue.mcfunction	ブルーシャードの神器レジストリ取得APIを追加

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+# 最後尾のsellとmaxUsesだけ移す
+    data modify storage asset:trader TempB[-1].sell set from storage asset:trader TempA[-1].sell
+    data modify storage asset:trader TempB[-1].maxUses set from storage asset:trader TempA[-1].maxUses
+
+# 売ってるのが神器の場合はIDを入れておく
+    execute if data storage asset:trader TempA[-1].sell.tag.TSB.ID run data modify storage asset:trader TempB[-1].sell set from storage asset:trader TempA[-1].sell.tag.TSB.ID
+


+# TempCに移す
+    data modify storage asset:trader TempC[-1] set from storage asset:trader TempB[-1]
+


+# 日替わり更新するか
+    execute if data storage asset:trader {DailyUpdate:true} run tag @s add Trader.DailyUpdate
 # 取引一覧


EllaCoat · 2026-05-22T08:57:07Z

+#   asset_manager:trader/common/update_recipe/by_progress/check_version.m
+
+# 取引一覧を抽出
+    function oh_my_dat:please


[MUST] (おそらく)

Suggested change

function oh_my_dat:please

function oh_my_dat:please

data modify storage asset:trader Trades set value []

私が読んだ感じの話だけどね、コイツってさてはリセットされてないな？
なにが起きるかと言うとですね……日替わり商人が初めて召喚された場合の話だけど、

function asset_manager:trader/register/construct/set_data でデータが登録

内部で function asset_manager:trader/common/update_recipe/by_progress/ が発火、当然 keep_sells/ も発火。

asset:trader Trades はおそらくリセットされていないので……後はおわかりですね？

ちょっとリセット抜けというか、歪な用いられ方をしている箇所が多いから、ここで初期化処理を挟む提案をするべ

EllaCoat · 2026-05-22T09:05:45Z

+# OhMyDat呼び出し
+    function oh_my_dat:please
+
+# 日替わり商人の場合は現在の商品は維持し、攻略度だけ反映する


[WANT]
そもそも論として、ネクサスローダー経由（かな？）で新規召喚した時に register が 2 回発火する設計自体が、まずいのでなにかしらの対策は入れたほうがいいかもー

新規召喚時は by_progress/ の代わりに update.mcfunction を直接呼ぶとか？　知らんけど

EllaCoat · 2026-05-22T09:44:36Z

+#   asset_manager:trader/common/update_recipe/by_daily
+#   asset_manager:trader/common/update_recipe/by_progress/

 # 攻略率を取得


[WANT]
select_artifact と、この update の mcf で api:global_vars/game_progress が重複呼び出しされているね。
まぁ別にそんなに影響はないけど、対応したかったら対応して、どうぞ。

EllaCoat · 2026-05-22T09:51:57Z

+    function api:artifact/get_registry/normal
+    function api:artifact/get_registry/hide


[NITS]
asset:artifact RarityRegistry って想定重い SNBT （要するに重いストレージ）で
current_progress/set_pool.m って単一 Rarity しか参照しなさそうだから、RarityRegistry レベル別に取得する際の API とか後で作ったら良さそう。

EllaCoat · 2026-05-22T09:55:42Z

+#> api:artifact/get_registry/red
+#
+#
+#
+# @api


[NITS]
説明を書け～～～～～～～～～～～～～～～～～～～～～
（他も同様だゾ）

Suggested change

#> api:artifact/get_registry/red

#

#

#

# @api

#> api:artifact/get_registry/red

#

# レッドシャードの神器プールを返す

#

# @output storage api: Return.Registry.Red : int[][] (神器IDの2次元配列、第1次元はRarity 0~4)

# @api

EllaCoat · 2026-05-22T10:14:43Z

Copilot の指摘の吟味はしてないけど、なんか超的外れって訳でもなさそうだから、見ておくと良いよ。

Lapis-LJA added 8 commits November 5, 2025 20:49

神器のプールを取得するAPIを作成

4177b2b

暫定的に途中まで日替わり更新をcore側に実装

04a905e

どうしたらregisterを呼び出せるか、というところまで

d0fdf10

暫定的にバージョンアップと日替わり更新の処理を分離

0781353

リセット漏れを修正

be88047

正常に日替わり更新されない問題を修正

82de11f

rare -> hide

513d8bf

完成

d0626ae

Lapis-LJA requested review from ChenCMD and haiiro2gou as code owners November 12, 2025 16:31

Lapis-LJA changed the title ~~✨ [Trader 15] 日替わり商人の実装 & 日替わり更新の実装~~ ✨ [Trader 15] 日替わり商人の実装 & 商人の日替わり更新システムの実装 Nov 12, 2025

Lapis-LJA marked this pull request as draft November 12, 2025 16:38

Lapis-LJA added 18 commits November 13, 2025 12:31

"レッドー"を修正

9c47f34

デバッグ用コマンドをコメントアウト

1fe7a42

デバッグ用コマンドをコメントアウト(1)

2da4fd8

MaxUsesを設定

3857414

Merge branch 'dev/lapis_daily_trader' of https://github.com/ProjectTS…

424afc1

…B/TheSkyBlessing into dev/lapis_daily_trader

攻略度によって正常に更新されない問題を修正

48c0386

HideRarityRegistryの初期化をしていない問題を修正

d5a19a9

正常に配列の結合が出来ていない問題を修正

3b6f526

デバッグコマンドがそのままな問題を修正

fc7c096

スコアの説明を修正

2499f00

売るアイテムを表示するために必要な攻略%を設定できるように

1418510

想定外の値の際にエラーを出力するように

ab8de54

表示に必要な攻略度を設定

60a6427

取引の諸々を設定

ad6aecc

売る神器の仕様を調整

7c5e853

売るアイテムを表示する設定をなかったことに

ae7f49f

エラーを修正

aaacb27

価格を調整

889cd29

Lapis-LJA marked this pull request as ready for review November 14, 2025 10:21

一部コメントを修正

cd40d9d

Lapis-LJA changed the title ~~✨ [Trader 15] 日替わり商人の実装 & 商人の日替わり更新システムの実装~~ ✨ [Trader 15] 日替わり商人の作成 & 商人の日替わり更新システムの実装 Nov 14, 2025

Lapis-LJA added 3 commits November 14, 2025 20:56

Temporaryが正常にリセットされない問題を修正

a632822

コメントなどを修正

e0c2bfd

IMP-Docにマクロの引数について定義

7f90bb0

Lapis-LJA added 3 commits December 23, 2025 11:53

Merge branch 'master' into dev/lapis_daily_trader

40e946b

global storageに設定されたシャードの解禁攻略度を使うように

501d632

リセット漏れを修正

492f63f

ChenCMD added the ⛏️ minor このPRはパッチアップデートに含められないよ label Dec 26, 2025

Lapis-LJA and others added 7 commits January 1, 2026 15:16

Merge branch 'master' into dev/lapis_daily_trader

66456cf

リセット漏れを修正

0ec6e6f

攻略段階の販売においてLv1は最初から購入できるように

9425091

Merge branch 'master' into dev/lapis_daily_trader

71af9d2

島攻略時に商品が更新されないように

adf0219

コメントを修正

1b51e16

売っているのが神器の場合はIDを入れるように

304b900

haiiro2gou requested a review from Copilot May 22, 2026 09:34

Copilot started reviewing on behalf of haiiro2gou May 22, 2026 09:35 View session

Copilot AI reviewed May 22, 2026

View reviewed changes

EllaCoat requested changes May 22, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

✨ [Trader 15] 日替わり商人の作成 & 商人の日替わり更新システムの実装#2169

✨ [Trader 15] 日替わり商人の作成 & 商人の日替わり更新システムの実装#2169
Lapis-LJA wants to merge 40 commits into
masterfrom
dev/lapis_daily_trader

Lapis-LJA commented Nov 12, 2025 •

edited

Loading

Uh oh!

Lapis-LJA commented Nov 24, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

EllaCoat May 22, 2026

Uh oh!

EllaCoat May 22, 2026

Uh oh!

EllaCoat May 22, 2026 •

edited

Loading

Uh oh!

EllaCoat May 22, 2026 •

edited

Loading

Uh oh!

EllaCoat May 22, 2026

Uh oh!

EllaCoat commented May 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		# TempCに移す
		data modify storage asset:trader TempC[-1] set from storage asset:trader TempB[-1]

	function oh_my_dat:please
	function oh_my_dat:please
	data modify storage asset:trader Trades set value []

		function api:artifact/get_registry/normal
		function api:artifact/get_registry/hide

Conversation

Lapis-LJA commented Nov 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Lapis-LJA commented Nov 24, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

EllaCoat May 22, 2026

Choose a reason for hiding this comment

Uh oh!

EllaCoat May 22, 2026

Choose a reason for hiding this comment

Uh oh!

EllaCoat May 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EllaCoat May 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EllaCoat May 22, 2026

Choose a reason for hiding this comment

Uh oh!

EllaCoat commented May 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Lapis-LJA commented Nov 12, 2025 •

edited

Loading

EllaCoat May 22, 2026 •

edited

Loading

EllaCoat May 22, 2026 •

edited

Loading