Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughDocumentation-only updates to Satellite-API: frontmatter set Changes
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
for-developers/Satellite-API.md (1)
281-283: Please clarify LOCATION index bases (page vs row/col).
PAGE/ROW/COLis introduced with examples, but row/column base (0-based vs 1-based) isn’t explicit here. A one-line clarification would prevent off-by-one client bugs.🧭 Suggested clarification
-A location is expressed as `PAGE/ROW/COL` using Companion's native location syntax, e.g. `1/2/3` for page 1, row 2, column 3. +A location is expressed as `PAGE/ROW/COL` using Companion's native location syntax. +`PAGE` is 1-based, while `ROW` and `COL` follow Companion's button coordinates (explicitly document whether these are 0-based or 1-based here), e.g. `1/2/3`.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 930c006d-2380-4c28-8135-55a5b16505fe
📒 Files selected for processing (1)
for-developers/Satellite-API.md
bitfocus/companion#4043
Summary by CodeRabbit
Documentation
New Features