fix: remove wikipedia-analysis from ASO paid onboarding profile#2567
Open
srobins wants to merge 2 commits into
Open
fix: remove wikipedia-analysis from ASO paid onboarding profile#2567srobins wants to merge 2 commits into
srobins wants to merge 2 commits into
Conversation
wikipedia-analysis is an LLMO-only audit per the Opportunity Type Catalog. It has no corresponding ASO opportunity type and should not be included in the ASO paid onboarding profile. Fixes SITES-45864 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
tkotthakota-adobe
approved these changes
Jun 5, 2026
toc is LLMO only per the Opportunity Type Catalog — no corresponding ASO opportunity type. Tracked in SITES-45864. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
This PR will trigger a patch release when merged. |
MysticatBot
approved these changes
Jun 15, 2026
There was a problem hiding this comment.
Hey @srobins,
Verdict: Approve - clean config removal, no issues.
Changes: Removes wikipedia-analysis and toc from the ASO paid onboarding profile's audit list (1 file).
Non-blocking (1): minor issues and suggestions
- nit: PR title says "remove wikipedia-analysis" but the diff also removes
toc; the body's "Ask" section framestocremoval as an open question yet the code already removes it - updating the title to mention both (e.g. "fix: remove wikipedia-analysis and toc from ASO paid onboarding profile") would keep git history accurate and avoid confusion for future readers -static/onboard/profiles.json:131
Skill: pr-review | Model: us.anthropic.claude-opus-4-6-v1[1m] | Duration: 0m 44s | Cost: $1.74 | Commit: e7ce5ae6f1968db7a5cc41dec5ff930a87c617d8
If this code review was useful, please react with 👍. Otherwise, react with 👎.
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
4 participants
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.
Summary
wikipedia-analysisfrom thepaidprofile instatic/onboard/profiles.jsonwikipedia-analysisis classified as LLMO only — it has no corresponding ASO opportunity typeAsk
tocwas recently added and also appears to be LLMO only in the same wiki page. Should we removetocalso?Test plan
paidprofile no longer includeswikipedia-analysisin the audit listpaidprofile🤖 Generated with Claude Code