Skip to content

Regenerate the Boost agent files - #84

Merged
simonhamp merged 2 commits into
mainfrom
chore/boost-update
Oct 24, 2026
Merged

simonhamp merged 2 commits into
mainfrom
chore/boost-update

Conversation

@simonhamp

Copy link
Copy Markdown
Member

This reruns php artisan boost:update --no-discover with 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/skills and a new config/boost.php change.

What changes for agents

  • CLAUDE.md no longer pins a list of package versions. It tells agents to check composer show instead.
  • search-docs is only required when a change depends on framework or package APIs, not before every edit.
  • New testing rules: write tests for behaviour changes, run the narrowest set while working, then ask for the full suite.
  • The NativePHP sections are much longer: nested child components, theme tokens and font aliases, mobile-ui element rules and the text input traps. Examples now use the Native\Mobile\UI namespace, which is what we have installed. They also use @tap, which is an alias for the @press the app uses everywhere, so existing views are fine.
  • pest-testing is replaced by testing-best-practices, and laravel-best-practices drops its testing rule to match.
  • nativephp-mobile-ui is new. It's the element reference for writing views.
  • infer-conventions is new. It only runs when someone invokes it by name.
  • nativephp-mobile, nativephp-webview-to-native and laravel-best-practices are refreshed.

Left out

nativephp-v3-to-v4-upgrade is excluded in config/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.exclude is the only way to keep one out.

Worth a look

  • Build commands: The old CLAUDE.md said never run native:run and always hand the command to the user. The new guidance, here and in the nativephp-mobile skill, says to run native:run ios <UDID> --no-tty -n yourself 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.
  • Project Rules: A new paragraph tells agents to read .ai/rules before editing. This repo has no .ai/rules, and the paragraph says to carry on without it in that case, so it does nothing today. Setting boost.rules.enabled to false removes it.
  • Ask for before and after screenshots on UI pull requests #83: The screenshots PR also touches CLAUDE.md and boost.json, so whichever merges second needs boost:update rerun 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, both ui-screenshots symlinks and AGENTS.md all come out right, and the excluded skill stays out of .agents/skills too.

Testing

php artisan test --compact passes (1196 tests). There's no app code in this PR, so nothing to check on a device.

🤖 Generated with Claude Code

simonhamp and others added 2 commits October 24, 2026 03:00
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
simonhamp marked this pull request as ready for review October 24, 2026 02:18
@simonhamp
simonhamp merged commit 64f4426 into main Oct 24, 2026
3 checks passed
@simonhamp
simonhamp deleted the chore/boost-update branch October 24, 2026 03:00
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>
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>
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.

1 participant