Skip to content

Integrate NJS-GROWTH-07 proof assets into README#14

Merged
SebaSOFT merged 3 commits into
mainfrom
growth/njs-growth-07-readme-docs-proof-assets
Jun 23, 2026
Merged

Integrate NJS-GROWTH-07 proof assets into README#14
SebaSOFT merged 3 commits into
mainfrom
growth/njs-growth-07-readme-docs-proof-assets

Conversation

@SebaSOFT

Copy link
Copy Markdown
Owner

Summary

  • embeds the accepted explainability trace SVG in the README as the current README-safe proof visual
  • links README and VitePress proof-asset docs to benchmark methodology, visual-proof guardrails, explainability metadata, and playground capture specification
  • adds/threads accepted NJS-GROWTH-07 proof asset docs, prompts, manifest, generated SVG/HTML assets, and the playground capture contract test

Guardrails

  • no benchmark result claims are introduced
  • benchmark visuals remain blocked until actual benchmark output sets result_kind: "actual_benchmark", is_placeholder: false, and claims_allowed: true
  • playground GIF capture remains explicitly blocked until the playground route and capture automation exist

Validation

  • git diff --check
  • yarn lint
  • yarn test — 12 files / 63 tests passed
  • yarn examples
  • yarn docs:build

Dependency note

This PR is stacked on PR #13 (growth/njs-growth-07-build-proof-assets-benchmarks-playground) because it integrates the accepted proof-asset bundle produced downstream of the benchmark contract work.

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request introduces a comprehensive visual proof system and benchmark asset plan for Neuron-JS, including storyboards, prompts, manifests, and specifications for benchmark charts, safety carousels, and playground captures. It also adds contract tests to validate these specifications and integrates the explainability trace diagram directly into the documentation and README. Feedback on the changes points out an unused ".card" CSS class within the newly added explainability-trace-diagram.svg stylesheet that should be removed for cleanliness.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread docs/benchmarks/assets/generated/explainability-trace-diagram.svg
Base automatically changed from growth/njs-growth-07-build-proof-assets-benchmarks-playground to main June 23, 2026 00:54
SebaSOFT and others added 2 commits June 22, 2026 21:56
Replaces 10 inline fill="#0f172a" rects with the existing .card class so
the previously-unused stylesheet rule is applied, removing duplication.
Output is visually identical. Resolves the code-review finding about the
dead .card class.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@SebaSOFT SebaSOFT merged commit d4f8c5a into main Jun 23, 2026
2 checks passed
@SebaSOFT SebaSOFT deleted the growth/njs-growth-07-readme-docs-proof-assets branch June 23, 2026 01:04
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