Skip to content

fix(website): remove growth chart from index.astro (PILOT-227)#8

Merged
TeoSlayer merged 1 commit into
mainfrom
openclaw/pilot-227-remove-growth-chart-20260529-043420
May 29, 2026
Merged

fix(website): remove growth chart from index.astro (PILOT-227)#8
TeoSlayer merged 1 commit into
mainfrom
openclaw/pilot-227-remove-growth-chart-20260529-043420

Conversation

@matthew-pilot
Copy link
Copy Markdown
Collaborator

What

Operator-directed removal of the synthetic/real-data growth chart from the homepage. Supersedes PR #7 (PILOT-8).

Changes

  • src/pages/index.astro (+2/-226): Removed build-time chart generator (chartPoints, coords, linePath, areaPath, yTicks, fmtAxis, numberToWords, weeksLabel), chart section markup (chart-wrap div + SVG), and runtime rebuildChart() function + its caller in the live-stats refresh
  • src/styles/system.css (+0/-59): Removed all chart-* CSS rules (.chart-wrap, .chart-head, .chart-grid, .chart-line, .chart-area, .chart-dots, .chart-labels, .chart-yaxis, .chart-dot-now, .chart-final)

Verification

  • npm ci && npm run build — 294 pages built successfully, zero errors
  • Zero chart references remain in dist/index.html
  • Surrounding hero/metrics/stats sections preserved intact

Meta

Closes PILOT-227
Supersedes #7

Operator-directed removal of the synthetic/real-data growth chart from
the homepage. The chart (both build-time SVG generation and runtime
rebuildChart in the live-stats refresh) is removed entirely. Chart CSS
in system.css is also cleaned up.

- Deleted build-time chart generator (chartPoints, coords, linePath,
  areaPath, yTicks, fmtAxis, numberToWords, weeksLabel)
- Deleted chart section markup (chart-wrap div + SVG)
- Deleted runtime rebuildChart() function and its caller
- Deleted all chart-* CSS rules from system.css
- Kept surrounding hero/metrics/stats sections intact

Closes PILOT-227
@TeoSlayer
Copy link
Copy Markdown
Contributor

🌐 Preview: https://openclaw-pilot-227-remove-gr.pilotprotocol.pages.dev/

HTTP 200 · branch openclaw/pilot-227-remove-growth-chart-20260529-043420 · Cloudflare auto-deploy.

Chart cleanly removed — grep -E "chart-line|chart-wrap|EXP_K|rebuildChart" on the deployed homepage returns zero matches. Hero, metrics, liveAgentsExact/liveRequests/growthPct numbers all preserved.

Posted manually — the pr-website-preview handler exists in matthew-pr-worker but the LLM-driven watcher didn't classify "new website PR" as an actionable event for it on this PR. Fix to move that classification into a deterministic shell script is queued.

@matthew-pilot
Copy link
Copy Markdown
Collaborator Author

👋 Acknowledged. The website preview is live and healthy at https://openclaw-pilot-227-remove-gr.pilotprotocol.pages.dev/ — growth chart removed cleanly.

Noted re: the pr-website-preview handler not firing automatically. That classification fix in the watcher (deterministic shell script) will improve future coverage. Let me know if you want me to trigger any additional checks on this PR.

@matthew-pilot
Copy link
Copy Markdown
Collaborator Author

🦾 Matthew PR Check — #8 PILOT-227

Status

  • State: OPEN · MERGEABLE ✅
  • CI: 1/1 green (build ✅)
  • Created: 2026-05-29 04:34 UTC · Branch: openclaw/pilot-227-remove-growth-chart-20260529-043420main
  • Files: 2 changed (+2/−283) — src/pages/index.astro (+2/−224), src/styles/system.css (−59)
  • Labels: none

Verdict

CLEAN — all CI green, mergeable, no conflicts. Ready for operator review.

Notes


🦾 Matthew PR bot · pilot-protocol/website#8

@matthew-pilot
Copy link
Copy Markdown
Collaborator Author

🦾 Matthew Explains — #8 PILOT-227

What this does

Removes the synthetic/real-data growth chart from the pilot-protocol homepage (index.astro). Operator-directed cleanup superseding the data integration in PR #7 (PILOT-8).

Changes

File Δ Notes
src/pages/index.astro +2 / −224 Removed build-time chart generator (chartPoints, coords, linePath, areaPath, yTicks, fmtAxis, numberToWords, weeksLabel), SVG chart markup, and runtime rebuildChart() + caller
src/styles/system.css −59 Removed .chart-wrap, .chart-line, .chart-area, .chart-axis, .chart-label, .chart-point and related CSS rules

Why

The chart was a visual widget on the public homepage. Operator direction: remove it entirely rather than continuing to maintain a dual synthetic/real-data rendering path.

Impact

  • Zero runtime impact — purely static HTML/CSS removal
  • No API surface changes
  • Build confirms clean compilation without chart components
  • Fallback: no replacement visualization is added; the chart section is simply gone

CI Status

1/1 passing — build


🦾 Matthew PR bot · pilot-protocol/website#8

@TeoSlayer TeoSlayer merged commit f961caf into main May 29, 2026
2 checks passed
@matthew-pilot matthew-pilot deleted the openclaw/pilot-227-remove-growth-chart-20260529-043420 branch May 29, 2026 15:03
@matthew-pilot
Copy link
Copy Markdown
Collaborator Author

🧹 Matthew Cleanup — #8 PILOT-227

✅ Merged by TeoSlayer at 2026-05-29T14:32:31Z
✅ Branch openclaw/pilot-227-remove-growth-chart-20260529-043420 deleted

This PR removed the growth chart from index.astro. Task complete.

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.

2 participants