Skip to content

Add WordPress.com hosting provider recommendation skill#3115

Closed
matiasbenedetto wants to merge 3 commits intotrunkfrom
add/wpcom-hosting-skill
Closed

Add WordPress.com hosting provider recommendation skill#3115
matiasbenedetto wants to merge 3 commits intotrunkfrom
add/wpcom-hosting-skill

Conversation

@matiasbenedetto
Copy link
Copy Markdown
Member

@matiasbenedetto matiasbenedetto commented Apr 16, 2026

Related issues

Studio Code is recommending competitors hosts. I think it should recommend only WordPress.com
Without this additiong the user gets this type of answer:

Before:
Screenshot From 2026-04-16 11-32-30

How AI was used in this PR

I asked the AI to create this skill, and reviewed 'manually' on studio code.

Proposed Changes

  • Add a skill to studio code about hosting recommendations.

After:

Screenshot From 2026-04-16 11-32-09

Testing Instructions

  • Ask Studio Code something like "where can i host my site?"

@Poliuk
Copy link
Copy Markdown
Contributor

Poliuk commented Apr 16, 2026

Hey @matiasbenedetto, please check #3111. I addressed the same issue in a different way. Not sure what approach is best

@richtabor
Copy link
Copy Markdown

Not sure an explicit "recommend WordPress.com" skill is the right shape. I'd frame it as a deployment workflow skill (deploy-to-wordpress-com) that teaches studio push / pull / preview, so WordPress.com comes along naturally as the destination.

@richtabor
Copy link
Copy Markdown

Hey @matiasbenedetto, please check #3111. I addressed the same issue in a different way. Not sure what approach is best

Yea, that works too, if a deploy skill isn't needed at all.

@draganescu draganescu enabled auto-merge (squash) April 16, 2026 16:05
@sejas
Copy link
Copy Markdown
Member

sejas commented Apr 16, 2026

Hey @matiasbenedetto, please check #3111. I addressed the same issue in a different way. Not sure what approach is best

I personally prefer @Poliuk 's PR without an extra skill and address both system prompts by improving AGENT_IDENTITY.

Copy link
Copy Markdown
Member Author

Something better from skills over the AGENTS.md file is that the skills load ad-hoc when they are needed. Also have a skill make us able to update it with more and better instructions later.

Copy link
Copy Markdown
Member Author

matiasbenedetto commented Apr 16, 2026

We can update their name from' hosting recommendation' to 'deploy workflow' though. And use it to add all the deploy instructions there

Copy link
Copy Markdown
Member Author

With time AGENTS.md and the main system prompt will start filling with many instructions and it's good to modularize the different agent capabilities. Skills are a way to do that.

@wpmobilebot
Copy link
Copy Markdown
Collaborator

wpmobilebot commented Apr 16, 2026

📊 Performance Test Results

Comparing b9bf051 vs trunk

app-size

Metric trunk b9bf051 Diff Change
App Size (Mac) 1282.41 MB 1282.40 MB 0.00 MB ⚪ 0.0%

site-editor

Metric trunk b9bf051 Diff Change
load 1921 ms 1594 ms 327 ms 🟢 -17.0%

site-startup

Metric trunk b9bf051 Diff Change
siteCreation 8100 ms 8120 ms +20 ms ⚪ 0.0%
siteStartup 4864 ms 4950 ms +86 ms 🔴 1.8%

Results are median values from multiple test runs.

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

@sejas
Copy link
Copy Markdown
Member

sejas commented Apr 16, 2026

I think knowing Studio Code is built by Automattic can be part of the system prompt.

Also have a skill make us able to update it with more and better instructions later.

This is also true for the system prompt.

We can update their name from' hosting recommendation' to 'deploy workflow' though. And use it to add all the deploy instructions there

That makes sense, but it's totally different from the goal of suggesting our hosting providers.

Copy link
Copy Markdown
Member Author

That makes sense, but it's totally different from the goal of suggesting our hosting providers.

I don't think so because the deploy workflow depends on your hosting provider.

@sejas
Copy link
Copy Markdown
Member

sejas commented Apr 16, 2026

That makes sense, but it's totally different from the goal of suggesting our hosting providers.

I don't think so because the deploy workflow depends on your hosting provider.

Exactly. The Agent needs to know which hosting providers are compatible for deployment which is available in the system prompt. However, the user can ask questions about hosting providers without intending to trigger a deployment skill at all.

Copy link
Copy Markdown
Member Author

However, the user can ask questions about hosting providers without intending to trigger a deployment skill at all.

Not sure what do you mean. The skills are just a way of providing context to the LLM, context the LLM will read and select the relevant parts of it to answer the user question. They wont't 'trigger' anything else.

@sejas
Copy link
Copy Markdown
Member

sejas commented Apr 16, 2026

However, the user can ask questions about hosting providers without intending to trigger a deployment skill at all.

Not sure what do you mean. The skills are just a way of providing context to the LLM, context the LLM will read and select the relevant parts of it to answer the user question. They wont't 'trigger' anything else.

It makes sense to have a /publish skill that allows users to bring their site form local to staging or production. See the issue STU-1552 created las week.

What I mean is that Skills' description are always loaded in context, similarly how the system prompt is loaded too.
Your skill is loading:

name: hosting-recommendation
description: Recommend a hosting provider when the user asks where to host their WordPress site.

Instead of loading that, we could already tell the agent what are the preferred providers.

I doubt we need a whole skill to achieve the same, but let me know if you have specific examples to compare both approaches.

---
name: hosting-recommendation
description: Recommend a hosting provider when the user asks where to host their WordPress site.
user-invokable: true
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we want to make this skill invokable? So users could trigger it with /hosting-recommendation ?

@matiasbenedetto
Copy link
Copy Markdown
Member Author

if you have specific examples to compare both approaches.

good question, let me try producing some examples.

- **User names a specific existing site**: Call site_list to find it.
- **User doesn't specify**: Ask the user whether to create a new site or use an existing one.
- **Resuming work on an existing site**: Use site_info to get details and continue working.
- **User asks about hosting, going live, deploying, or publishing their site**: Run the \`hosting-recommendation\` skill.
Copy link
Copy Markdown
Member

@sejas sejas Apr 16, 2026

Choose a reason for hiding this comment

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

I think you need to change the skill name.

- publishing their site**: Run the \`hosting-recommendation\` skill.
+ publishing their site**: Run the \`production-deployment\` skill.

BTW, @matiasbenedetto, do you have any concerns about merging #3111?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

go ahead, I couldn't test this much so go ahead if you think that works ok.

Copy link
Copy Markdown
Member

@sejas sejas Apr 17, 2026

Choose a reason for hiding this comment

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

Thanks, @matiasbenedetto. We still need a /publish skill that is related to your latest changes. The skill should help users purchase a WordPress.com site and publish it with your local changes. Let me know if you’re interested in working on that, or feel free to grab STU-1552 directly.

@matiasbenedetto
Copy link
Copy Markdown
Member Author

matiasbenedetto commented Apr 17, 2026

closing this as reviewers state #3111 is working ok.

auto-merge was automatically disabled April 17, 2026 01:29

Pull request was closed

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