Skip to content

chore: make this package avaiable with searching LAC - #113

Merged
Limitex merged 2 commits into
Limitex:mainfrom
anatawa12:patch-1
Aug 21, 2026
Merged

chore: make this package avaiable with searching LAC#113
Limitex merged 2 commits into
Limitex:mainfrom
anatawa12:patch-1

Conversation

@anatawa12

@anatawa12 anatawa12 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

🔍 What type of PR is this?

  • 🐛 Bug fix
  • ✨ New feature
  • 📖 Documentation update
  • 🎨 Refactoring
  • 🧪 Test
  • ⏪ Revert
  • 📝 Other

📝 Description

This should make tools supports keywords (like ALCOM) shows this package when searching with "LAC".

🔗 Related Issue

N/A

✅ Changes

chore: make this package avaiable with searching LAC

⚠️ Breaking Changes

None

🧪 Testing

Please describe the tests that you ran to verify your changes.

  • Confirmed it works in Unity 2022.3.22f1
  • Confirmed package import via VRChat Creator Companion
  • Verified texture compression works correctly on test avatars

📸 Screenshots / Videos

If your changes have a visual component, please provide screenshots or videos.

💌 To the Reviewers

Please provide any specific feedback or areas you would like the reviewers to focus on.

I hope changelog changes are not nessesary

Additional Context

Add any other context about the pull request here.

Summary by CodeRabbit

  • Chores
    • Added the lac keyword to package metadata for improved discoverability.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ed8712c6-c49b-4b63-aeee-3ce4dfe64fb1

📥 Commits

Reviewing files that changed from the base of the PR and between 0deca30 and 349ca93.

📒 Files selected for processing (1)
  • package.json

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The package metadata now includes the lac keyword.

Changes

Package metadata

Layer / File(s) Summary
Add package keyword
package.json
Adds lac to the package keywords.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 349ca

This is a narrowly scoped package discovery change with no actionable merge-blocking risk remaining after normal checks and review.

Poem

A rabbit spots a keyword bright,
“lac” now hops into metadata light.
One small change, neat and clear,
The package grows a new idea here.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title clearly identifies the metadata change that enables package discovery when users search for LAC.
Description check ✅ Passed The description covers the required sections and explains the change, but it reports no completed testing.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

Comment thread package.json
@Limitex

Limitex commented Aug 20, 2026

Copy link
Copy Markdown
Owner

@anatawa12
Thanks for the PR! I'll merge once CI is green.

@anatawa12

Copy link
Copy Markdown
Contributor Author

the pull_request_target workflows are not my problem, repository owner's action are required. For changelog, As I noted in PR Description, i think no changelog entru is not necessary so not updated.

@Limitex

Limitex commented Aug 20, 2026

Copy link
Copy Markdown
Owner

@anatawa12
Yeah, I'm planning to merge this PR after fixing the CI issues in #114.

@Limitex
Limitex merged commit 8de772c into Limitex:main Aug 21, 2026
4 of 5 checks passed
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