Repository navigation
Address important live README review findings - #7
Merged
Merged
Conversation
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
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.
Scope
Refines the existing Install route-button and compact-page-banner work, and addresses the important findings from Claude's live GitHub README review. Four participating repositories only: snode.c, mqttsuite, AISuite and CodexUI; .github owns the shared frame and generated organization profile. Not merged.
Important review fixes
Verification
python3 tools/snodec_sync check,python3 -m unittest discover -s tests -v(33 passed), and generated-profile drift check passed in .github.Accounting for this review-fix follow-up
Production-source physical lines: +8/-10 (existing profile renderer +4/-4; obsolete CMake comment corrected +4/-6, executable CMake byte-for-byte equivalent after removing comments/blank lines). No runtime/build behavior change and no production-code growth. Test source: +37/-7, preserving original checks and adding compact-status regression coverage. Documentation/static assets: +1266/-1614; historical record extraction reuses the existing banner assets.
Claude's main report and visual supplement informed this change; its full completion supplement hit its session quota. This is not a claim of fresh Claude approval. Optional root-banner/naming redesign and CI/runtime repairs are outside this scoped follow-up.
This repository
Rendered preview. Follow its menu links through the documentation system.
Review-fix accounting here: {"production": {"added": 4, "deleted": 4}, "tests": {"added": 37, "deleted": 7}, "documentation_and_assets": {"added": 58, "deleted": 57}}
Final mobile check
Long repository/kind labels receive GitHub-supported zero-width break opportunities in display text only; catalog identity and canonical URLs are unchanged. This fixes the remaining phone-width overflow without repository-specific exceptions. The actual published organization preview was visually checked at 390 px. The renderer retains one identity authority, and no workflow behavior changes.