Skip to content

fix: prevent artifacts muffling in display cases - #38

Merged
Drefvelin merged 1 commit into
mainfrom
fix/artifact-case-muffle
Oct 1, 2026
Merged

Drefvelin merged 1 commit into
mainfrom
fix/artifact-case-muffle

Conversation

@Drefvelin

Copy link
Copy Markdown
Contributor

Summary

  • Update artifact care timestamps when artifacts enter or leave display furniture.
  • Treat time inside display furniture as housed, so artifacts recover from muffle instead of accumulating it while stored.
  • Add event coverage for the display case transitions.

Validation

  • mvn -B --no-transfer-progress clean verify -DskipTests=false -Dmaven.test.skip=false — passed (443 tests and JaCoCo coverage gate).
  • The repository has no dev branch; this PR targets its only branch, main.

@coderabbitai

coderabbitai Bot commented Oct 1, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 9b275fb2-2500-473f-8de2-3ae82d778ef2

📥 Commits

Reviewing files that changed from the base of the PR and between 2bb5b08 and 6760b13.

📒 Files selected for processing (2)
  • src/main/java/net/tfminecraft/magic/listener/ArtifactListener.java
  • src/test/java/net/tfminecraft/magic/ArtifactListenerTest.java
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@Drefvelin
Drefvelin merged commit 17d4112 into main Oct 1, 2026
1 of 2 checks passed
@Drefvelin
Drefvelin deleted the fix/artifact-case-muffle branch October 1, 2026 07:48
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.

2 participants