Skip to content

Conversation

@MrKono
Copy link
Member

@MrKono MrKono commented Jan 26, 2026

GTECore製Slice'N'Spliceの仕様を、本来のSlice'N'Spliceの仕様に近づけました。

  • 1スロットにつき1アイテムしか搬入できない
  • レシピに使うものしか入らない
  • レシピ通りじゃないと動かない (ALと同じ)

また、これらを説明するtooltipを追加しました

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @MrKono, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

このプルリクエストは、GTECoreのSlice'N'Splice機械の動作を、より厳格な入力規則とレシピ検証を導入することで、その本来の設計意図に合わせることを目的としています。これにより、機械のアイテム処理とレシピ実行のロジックが強化され、ユーザーはより正確な操作が求められるようになります。変更は、専用のMetaTileEntityの実装と、新しい動作を説明するローカライズされたツールチップの追加によって実現されています。

Highlights

  • Slice'N'Spliceの動作仕様変更: GTECore製Slice'N'Spliceの動作が、本来の仕様に近づけられました。これにより、各スロットには1つのアイテムしか配置できなくなり、レシピに合致するアイテムのみが受け入れられ、レシピ通りの順番でアイテムが配置されていないと機械が動作しなくなります。
  • カスタムMetaTileEntityの実装: Slice'N'Spliceの新しい動作をサポートするため、MetaTileEntitySliceNSpliceという新しいクラスが追加されました。このクラスは、カスタムのインベントリハンドリングとレシピロジックを導入し、アイテムの挿入とレシピの検証に厳格なルールを適用します。
  • ツールチップの追加: 新しい動作仕様をユーザーに明確に伝えるため、Slice'N'Spliceのツールチップに説明文が追加されました。これにより、各スロットのアイテム制限、レシピ関連アイテムのみの受け入れ、およびレシピ順序の要件がユーザーインターフェースで確認できるようになります。
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

このプルリクエストは、Slice'N'Spliceマシンの仕様をオリジナルのEnder IOのものに近づけるための変更ですね。アイテム投入の制限やレシピの厳密なチェックが追加され、より特徴的なマシンになったと思います。
コード全体はうまく実装されていますが、いくつか修正すべき点と改善の提案があります。

  • createMetaTileEntityメソッドで誤ったテクスチャが使用されています。
  • isItemValidメソッドのパフォーマンスを改善する提案をしました。
  • スタイルガイドに準拠していないコメントがいくつか見つかりましたので、修正を提案しています。
    これらの点を修正することで、より品質の高いコードになるでしょう。

@MrKono MrKono requested a review from tier940 January 26, 2026 13:52
@tier940 tier940 changed the title Slice'N'Spliceの仕様を変更 Slice'N'Splice spec change Jan 27, 2026
@tier940 tier940 merged commit 395e92c into master Jan 27, 2026
2 checks passed
@tier940 tier940 deleted the kono/addSliceNSpliceLogic branch January 27, 2026 06:52
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.

3 participants