Skip to content

skills: refresh elegant-reports and add smoke test - #17

Merged
jdrhyne merged 1 commit into
mainfrom
feat/elegant-reports-clean-clone-smoke
Mar 26, 2026
Merged

skills: refresh elegant-reports and add smoke test#17
jdrhyne merged 1 commit into
mainfrom
feat/elegant-reports-clean-clone-smoke

Conversation

@jdrhyne

@jdrhyne jdrhyne commented Mar 26, 2026

Copy link
Copy Markdown
Owner

Summary

Refresh skills/elegant-reports from the cleaned public clone and add a minimal verification path so the skill can be merged/published without carrying local-machine baggage.

What actually changed

  • replaced the older elegant-reports repo copy with the cleaned public version
  • removed machine-specific/auth-loading behavior from generate.js
    • removed the fallback read from ~/clawd-nuri-internal/config/mcporter.json
    • removed the hard-coded ~/nuri_workspace/... sandbox path
    • now uses os.tmpdir() for sandbox output and requires NUTRIENT_DWS_API_KEY explicitly via env
  • cleaned skill metadata/docs
    • removed Nuri-specific author/examples
    • replaced local paths like ~/clawd-nuri-internal/... with generic instructions
    • moved design research into references/nordic-design-research.md
  • removed checked-in generated output artifacts
    • deleted examples/sample-output.html
    • deleted examples/sample-output.pdf
  • added a tiny smoke test at skills/elegant-reports/scripts/smoke-test.js
    • verifies every template/theme pair loads and renders a full HTML document
    • does not require a live Nutrient API key
  • added CI workflow .github/workflows/elegant-reports-smoke.yml
    • runs on PRs/pushes that touch skills/elegant-reports/**
    • installs dependencies and runs npm test
  • refreshed supporting templates/theme assets included in the clean clone

Why

The previous checked-in version still contained private/local environment assumptions (mcporter, clawd-nuri-internal, nuri_workspace) and generated sample artifacts. This PR makes the published skill safer and cleaner to clone, while adding a lightweight non-network validation step.

Validation

  • npm --prefix skills/elegant-reports test
  • node skills/elegant-reports/generate.js --list
  • hygiene grep for local/private refs in skills/elegant-reports
  • npm pack --dry-run from skills/elegant-reports

@jdrhyne
jdrhyne merged commit 2d7a3c9 into main Mar 26, 2026
1 check passed
@jdrhyne
jdrhyne deleted the feat/elegant-reports-clean-clone-smoke branch March 26, 2026 22:55
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