fix(fletchit.lic): v2.2.1 enhance TRASH messaging

[mrhoribu]

Updated version to 2.2.1 and added new TRASH response messaging.

Summary by CodeRabbit

• Bug Fixes

  • Improved trash-receptacle detection in learning mode shaft disposal by expanding recognition of success responses.

• Chores

  • Bumped version to 2.2.1 and updated changelog.

[coderabbitai]

Warning

Rate limit exceeded

@mrhoribu has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 57 minutes and 7 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 10ad8ed9-69a4-4113-b21c-a8f9e56d095c

📥 Commits

Reviewing files that changed from the base of the PR and between 75470eb and c418418.

📒 Files selected for processing (1)

• scripts/fletchit.lic

📝 Walkthrough

Walkthrough

This pull request bumps the FletchIt script version from 2.2.0 to 2.2.1 and improves learning-mode shaft disposal by recognizing an additional TRASH success response phrase ("you feel a moment of personal pride") when detecting trash-receptacle availability.

Changes

Trash Detection Enhancement

Layer / File(s)	Summary
Trash Detection Logic scripts/fletchit.lic	In finalize_arrow learning-mode shaft disposal, expanded trash-receptacle detection to include an additional TRASH success response phrase when computing trash_can_available?.
Version and Changelog Update scripts/fletchit.lic	Version bumped to 2.2.1 with changelog entry noting "Add additional TRASH response messaging".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: a version bump to 2.2.1 and enhancement to TRASH response messaging detection in the fletchit.lic script.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/fletchit-v2.2.1

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

scripts/fletchit.lic (2)

224-224: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Update FletchIt::Config::VERSION to 2.2.1 to match this release.

Line 224 still reports 2.2.0, so help/settings output will show the wrong version even after this PR.

Proposed fix

-    VERSION = "2.2.0" unless defined?(FletchIt::Config::VERSION)
+    VERSION = "2.2.1" unless defined?(FletchIt::Config::VERSION)

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@scripts/fletchit.lic` at line 224, Update the embedded default VERSION string
from "2.2.0" to "2.2.1" so FletchIt::Config::VERSION reflects the new release;
specifically modify the assignment line that currently reads VERSION = "2.2.0"
unless defined?(FletchIt::Config::VERSION) to use "2.2.1".

---

698-698: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Replace non-ASCII dashes in comments to unblock Rubocop.

CI is currently failing on Style/AsciiComments for these lines.

Proposed fix

-      # In learning mode, we only need wood — no glue, fletchings, paint, or paintsticks
+      # In learning mode, we only need wood - no glue, fletchings, paint, or paintsticks
...
-      # For learning mode — try trash can first, fall back to drop
+      # For learning mode - try trash can first, fall back to drop
...
-          # Unknown room — test for trash can
+          # Unknown room - test for trash can

Also applies to: 1259-1259, 1266-1266

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@scripts/fletchit.lic` at line 698, Replace the non-ASCII em-dash characters
in comment text with ASCII hyphens: locate comments containing the em-dash, e.g.
the comment string "# In learning mode, we only need wood — no glue, fletchings,
paint, or paintsticks" and the other similar comment occurrences, and change "—"
to "-" (or " - ") so the comments only contain ASCII characters to satisfy
Style/AsciiComments; no code logic changes required.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@scripts/fletchit.lic`:
- Line 224: Update the embedded default VERSION string from "2.2.0" to "2.2.1"
so FletchIt::Config::VERSION reflects the new release; specifically modify the
assignment line that currently reads VERSION = "2.2.0" unless
defined?(FletchIt::Config::VERSION) to use "2.2.1".
- Line 698: Replace the non-ASCII em-dash characters in comment text with ASCII
hyphens: locate comments containing the em-dash, e.g. the comment string "# In
learning mode, we only need wood — no glue, fletchings, paint, or paintsticks"
and the other similar comment occurrences, and change "—" to "-" (or " - ") so
the comments only contain ASCII characters to satisfy Style/AsciiComments; no
code logic changes required.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ae989839-cd4c-475d-8c83-ab54f9e08aa9

📥 Commits

Reviewing files that changed from the base of the PR and between 811ec91 and 75470eb.

📒 Files selected for processing (1)

• scripts/fletchit.lic