Skip to content

fix(performance): simplify sprites-report prerequisites#6

Open
igoramf wants to merge 6 commits into
mainfrom
feat/html-cache-investigation-skill
Open

fix(performance): simplify sprites-report prerequisites#6
igoramf wants to merge 6 commits into
mainfrom
feat/html-cache-investigation-skill

Conversation

@igoramf

@igoramf igoramf commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace hardcoded ClickHouse credentials in sprites-report prerequisites with a plain note to connect to the deco Stats Lake
  • Remove html-cache-report-cloudflare and html-cache-weekly-report-cloudflare skills (superseded by the unified html-cache-report)
  • Clean up design spec doc

Test plan

  • Verify sprites-report skill loads correctly and prerequisites section is clear

Summary by cubic

Adds new performance skills for HTML cache investigation, unified HTML cache reporting, loader cache reporting, and sprites reporting, and links them in the performance agent. Simplifies sprites-report prerequisites by removing hardcoded ClickHouse credentials and removes Cloudflare-only HTML cache report skills.

  • New Features

    • html-cache-investigation — find the first reason specific HTML pages don’t cache (headers, Set-Cookie, runtime/cache gates).
    • html-cache-report — unified CDN HTML cache report using ClickHouse Stats Lake.
    • loader-cache-report — analyze loader cache (hit/miss/stale, latency) via HyperDX and audit loader configs.
    • sprites-report — analyze /sprites.svg traffic and cache performance across sites.
  • Refactors

    • Replaced hardcoded ClickHouse creds in sprites-report with a simple note to connect to the deco Stats Lake.
    • Removed html-cache-report-cloudflare and html-cache-weekly-report-cloudflare (superseded by html-cache-report).
    • Updated README.md and agents/performance/agent.md to include the new skills.

Written for commit cf486a9. Summary will update on new commits.

igoramf added 6 commits March 4, 2026 12:45
…oudflare-specific html cache skills

Replace hardcoded ClickHouse credentials in sprites-report with a plain note to connect to the deco Stats Lake. Remove html-cache-report-cloudflare and html-cache-weekly-report-cloudflare skills (superseded by the unified html-cache-report). Clean up design spec doc.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 issue found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="agents/performance/agent.md">

<violation number="1" location="agents/performance/agent.md:34">
P2: This reintroduces `html-cache-report-cloudflare` even though the PR says it was superseded by `html-cache-report`; remove it to keep the skill list consistent with the intended consolidation.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Use this skill to analyze CDN traffic for `/sprites.svg` using ClickHouse Stats Lake. Reports which sites consume the most requests and bandwidth via sprites, and details cache performance per site.

- **html-cache-report-cloudflare.md**
Use this skill to generate an HTML cache report scoped exclusively to Cloudflare CDN sites. Uses GraphQL to discover which sites are on Cloudflare, ClickHouse for CDN cache metrics, and HyperDX to correlate loader cache blockers.

@cubic-dev-ai cubic-dev-ai Bot Apr 2, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2: This reintroduces html-cache-report-cloudflare even though the PR says it was superseded by html-cache-report; remove it to keep the skill list consistent with the intended consolidation.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At agents/performance/agent.md, line 34:

<comment>This reintroduces `html-cache-report-cloudflare` even though the PR says it was superseded by `html-cache-report`; remove it to keep the skill list consistent with the intended consolidation.</comment>

<file context>
@@ -17,4 +17,18 @@ Use this skill to improve how images are being used in the website, you will che
+Use this skill to analyze CDN traffic for `/sprites.svg` using ClickHouse Stats Lake. Reports which sites consume the most requests and bandwidth via sprites, and details cache performance per site.
+
+- **html-cache-report-cloudflare.md**
+Use this skill to generate an HTML cache report scoped exclusively to Cloudflare CDN sites. Uses GraphQL to discover which sites are on Cloudflare, ClickHouse for CDN cache metrics, and HyperDX to correlate loader cache blockers.
</file context>
Fix with Cubic

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