Skip to content

Fix /taxonomist wasm paths#2975

Open
mtias wants to merge 7 commits intotrunkfrom
fix/taxonomist-wasm-paths
Open

Fix /taxonomist wasm paths#2975
mtias wants to merge 7 commits intotrunkfrom
fix/taxonomist-wasm-paths

Conversation

@mtias
Copy link
Copy Markdown
Member

@mtias mtias commented Apr 4, 2026

Use positional args instead of env vars for taxonomist PHP scripts

mtias and others added 5 commits April 3, 2026 12:11
Move the full taxonomist skill content and PHP scripts into the CLI
plugin directory, removing the external skills/ folder. Register the
skill as a user-invokable slash command and add an install_taxonomy_scripts
tool so the agent can deploy scripts to target sites.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Environment variables don't work inside PHP WASM since there's no
shell to set them and the wp_cli MCP tool passes command-line args
directly. Switch all scripts to accept paths as WP-CLI eval-file
positional arguments ($args) with sensible defaults under the site's
taxonomist-data/ directory. Relative paths are resolved against
ABSPATH so they work correctly inside the WASM virtual filesystem.
@mtias mtias mentioned this pull request Apr 4, 2026
1 task
@wpmobilebot
Copy link
Copy Markdown
Collaborator

wpmobilebot commented Apr 4, 2026

📊 Performance Test Results

Comparing 006b45b vs trunk

app-size

Metric trunk 006b45b Diff Change
App Size (Mac) 1282.42 MB 1282.42 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 006b45b Diff Change
load 1841 ms 1904 ms +63 ms 🔴 3.4%

site-startup

Metric trunk 006b45b Diff Change
siteCreation 8136 ms 8123 ms 13 ms ⚪ 0.0%
siteStartup 4304 ms 4313 ms +9 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

Copy link
Copy Markdown

@martinremy martinremy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. This worked on my test blog and cleared up those messages.

Image

The scripts are working, but there are a couple of odd behaviors.

  • In this round, it assigned the parent category (Travel) in addition to the nested one (e.g. Africa). I think this is OK, might be good for SEO.
  • It keeps changing my default category to Music, a topic that accounts for only 20% of my posts. This one would be easy to squash with a line in SKILL.md.

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.

5 participants