Repository navigation
Regenerate the Boost agent files - #84
Merged
Merged
Conversation
Ran php artisan boost:update with Boost 2.10.3. CLAUDE.md and the vendored skills under .claude/skills now match the current Laravel, Pest and NativePHP guidelines. pest-testing is replaced by testing-best-practices, and infer-conventions and nativephp-mobile-ui are new. config/boost.php is published so the nativephp-v3-to-v4-upgrade skill can be excluded. The app is already on v4, and that skill would otherwise load whenever a screen renders blank or in the system font. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
simonhamp
marked this pull request as ready for review
October 24, 2026 02:18
simonhamp
added a commit
that referenced
this pull request
Oct 24, 2026
Merging #83 into #84 by hand skipped the boost:update rerun, so AGENTS.md was still #83's older copy and .agents/skills only held ui-screenshots. This reruns it on dev-main of nativephp/mobile, which also adds the nativephp-agent-loop skill (agents drive native:run, screenshots and the device log on a simulator themselves) and points the nativephp-mobile skill at it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 task done
simonhamp
added a commit
that referenced
this pull request
Oct 24, 2026
…#85) * Use mobile-air and mobile-ui main again The bounded gesture area work is merged (NativePHP/mobile-air#603 and NativePHP/mobile-ui#173), so the app goes back to requiring dev-main of both, with the same 4.6.0 and 0.8.0 aliases. The dev-main heads have the same tree as the feature branches #70 locked to, so no code changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * Catch the Boost agent files up with main Merging #83 into #84 by hand skipped the boost:update rerun, so AGENTS.md was still #83's older copy and .agents/skills only held ui-screenshots. This reruns it on dev-main of nativephp/mobile, which also adds the nativephp-agent-loop skill (agents drive native:run, screenshots and the device log on a simulator themselves) and points the nativephp-mobile skill at it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This reruns
php artisan boost:update --no-discoverwith Boost 2.10.3. The agent files were last generated on 30 September and upstream guidance has moved a lot since. Only CLAUDE.md, boost.json,.claude/skillsand a newconfig/boost.phpchange.What changes for agents
composer showinstead.search-docsis only required when a change depends on framework or package APIs, not before every edit.Native\Mobile\UInamespace, which is what we have installed. They also use@tap, which is an alias for the@pressthe app uses everywhere, so existing views are fine.pest-testingis replaced bytesting-best-practices, andlaravel-best-practicesdrops its testing rule to match.nativephp-mobile-uiis new. It's the element reference for writing views.infer-conventionsis new. It only runs when someone invokes it by name.nativephp-mobile,nativephp-webview-to-nativeandlaravel-best-practicesare refreshed.Left out
nativephp-v3-to-v4-upgradeis excluded inconfig/boost.php. The app is already on v4, and the skill's description would load it whenever a screen renders blank or in the system font, which would send an agent into a migration guide.Removing a skill from boost.json doesn't stick. Boost rewrites that list from whatever it discovers, so
boost.skills.excludeis the only way to keep one out.Worth a look
native:runand always hand the command to the user. The new guidance, here and in the nativephp-mobile skill, says to runnative:run ios <UDID> --no-tty -nyourself against a booted simulator, and only ask first for--watch, release builds, signing and store uploads. If we want to keep the old rule, it needs a custom guideline on top. I've left that out of this PR..ai/rulesbefore editing. This repo has no.ai/rules, and the paragraph says to carry on without it in that case, so it does nothing today. Settingboost.rules.enabledto false removes it.boost:updatererun after rebasing. I tried merging Ask for before and after screenshots on UI pull requests #83 on top of this locally. boost.json conflicts on the skill list. After keeping both sides and rerunning, the screenshots section, bothui-screenshotssymlinks and AGENTS.md all come out right, and the excluded skill stays out of.agents/skillstoo.Testing
php artisan test --compactpasses (1196 tests). There's no app code in this PR, so nothing to check on a device.🤖 Generated with Claude Code