Skip to content

Call本体機能の追加 #794

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 28 commits into
base: releases/1-21-1
Choose a base branch
from
Draft

Call本体機能の追加 #794

wants to merge 28 commits into from

Conversation

NePonpon
Copy link
Member

チケット URL

GH-790

対応内容・対応背景・妥協点

  • Callシステムをai空間へ移動し実装する。
  • skill:enemy/にあったスキル実装をai空間へ移動する。

やったこと

  • functions -> function
  • enemy:ai/call/ -> ai:/call/
  • skill:enemy/ -> ai:skill/

やってないこと

  • 新しいストレージ構造へ対応していない
  • Settingsをマクロ化し複雑なexecute機構を削除する
  • Conditionの呼び出しをマクロにする
  • Call Skillの呼び出しをマクロにする

テスト

  • 任意のMobのAIが正しく動作する
    • ただし、Functionを使っていてその中身が更新されていない場合の動作は保証されません

レビュー観点

  • 今までの使用感と同じようにMobAIが動作するか?

チェックリスト (ガイドライン)

@NePonpon NePonpon added 優先度: 高い レビュープリーズ レビューほしいときに使う(PRの場合はOpenAI 4oによるレビューが始まります) 1-21-1 MCver1.21.1で確認された問題 labels Apr 23, 2025
@NePonpon NePonpon self-assigned this Apr 23, 2025
@NePonpon NePonpon linked an issue Apr 23, 2025 that may be closed by this pull request
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@waxsd100 waxsd100 requested a review from Copilot May 3, 2025 14:30
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements the call functionality by migrating Call‐system components to the AI space and updating corresponding identifiers.

  • Updated GitHub workflow YAML configuration for improved multi‐line formatting.
  • Renamed skill identifiers from legacy “skill:enemy/” prefixes to the new “ai:skill/” pattern.
  • Minor formatting adjustments in the YAML file to enhance clarity and consistency.
Files not reviewed (19)
  • data/ai/function/call/call.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/check_block.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/check_nbt.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/function.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/hp.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/min_hp.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/min_lv.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/min_mp.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/mp.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/random_chance.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/target/.mcfunction: Language not supported
  • data/ai/function/call/condition/condition/target/half.mcfunction: Language not supported
  • data/ai/function/call/condition/logic/.mcfunction: Language not supported
  • data/ai/function/call/condition/logic/and.mcfunction: Language not supported
  • data/ai/function/call/condition/logic/or.mcfunction: Language not supported
  • data/ai/function/call/condition/logic/single.mcfunction: Language not supported
  • data/ai/function/call/execute/.mcfunction: Language not supported
  • data/ai/function/call/execute/all.mcfunction: Language not supported
Comments suppressed due to low confidence (1)

.github/workflows/chatgpt-review.yml:136

  • [nitpick] The renaming to the 'ai:skill' prefix enhances consistency; please verify that this update is reflected uniformly across relevant documentation and references.
-              - `ai:skill/laser/loop/security_laser`

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-21-1 MCver1.21.1で確認された問題 レビュープリーズ レビューほしいときに使う(PRの場合はOpenAI 4oによるレビューが始まります) 優先度: 高い
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature] aiのCall本体機能
2 participants