Repository navigation
docs(readme): landing visual + fix live-demo link + top.gg badge - #1821
Merged
Merged
Conversation
- Fix broken live-demo link (href was lucassantana.tech, missing lucky. subdomain) - Add top.gg listing badge (bot 962198089161134131) - Add landing feature screenshot to the Features section (public landing, no auth) - Record 2026-07-12 brand + presence ADRs (mascot keep-original, logo unify-to-cat) Hero shot held pending #1818 (landing says Apache 2.0 but project is ISC). Refs #1818 #1819 #1820
📝 WalkthroughWalkthroughREADME presence links and badges are updated. Two decision records document brand-asset regeneration outcomes, logo packaging, visual-proof plans, verified plumbing issues, and conditions for revisiting related work. ChangesPresence and branding
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
Failed to generate code suggestions for PR |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@decisions/2026-07-12-brand-asset-regen-tooling.md`:
- Around line 3-4: Update the ADR header’s Status field to reflect the completed
pilot and final decision documented later in the file, removing the obsolete
account-funding gate and “Open gate” reference while preserving the accepted
outcome.
In `@decisions/2026-07-12-readme-presence-visual-proof.md`:
- Around line 27-31: Align the decision record with the implemented screenshot
scope: update the README screenshot commitment to reflect only
assets/screenshots/landing-features.png and explicitly note that the hero and
other deferred screenshots are not included, or add the missing hero and
dashboard screenshots plus their README captions to satisfy the existing 2–3
screenshot commitment.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: b451edad-fe67-4228-bea8-236d418be99a
⛔ Files ignored due to path filters (1)
assets/screenshots/landing-features.pngis excluded by!**/*.png
📒 Files selected for processing (3)
README.mddecisions/2026-07-12-brand-asset-regen-tooling.mddecisions/2026-07-12-readme-presence-visual-proof.md
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 4 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
LucasSantana-Dev
enabled auto-merge (squash)
July 13, 2026 00:19
top.gg badge used logo=discord (Discord icon on a top.gg link); switch to logo=topdotgg. Update the brand-asset decision status from a stale funding gate to the completed pilot, and record the delivered single-screenshot scope in the presence decision record (coderabbit/cubic P3).
There was a problem hiding this comment.
0 issues found across 3 files (changes from recent commits).
Auto-approved: Documentation and cosmetic updates: README screenshot, link fix, badge, and decision docs. No business logic, security, or infrastructure changes.
Re-trigger cubic
|
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
1 participant
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.
What
Presence pass from the 2026-07-12
/research-and-decideon first-impression (README was text/table-strong but had zero product visuals):hrefpointed tohttps://lucassantana.tech(missing thelucky.subdomain) while the visible text saidlucky.lucassantana.tech.top.gg/bot/962198089161134131).Held / follow-ups
LICENSE,package.json). Won't embed a wrong-license screenshot; add it once the landing license text is corrected.Decision record
decisions/2026-07-12-readme-presence-visual-proof.md(+ the mascot/logo ADR2026-07-12-brand-asset-regen-tooling.md).Honest framing: this is cheap, obvious polish — not a validated growth lever.
Summary by cubic
Improved the README’s first impression with a landing screenshot, fixed the live demo link, and added a
top.ggbadge with correct branding. Also updated decision docs to reflect the pilot outcome.New Features
top.ggbadge linking to the live bot listing.Bug Fixes
logo=topdotggfor accurate branding.Written for commit e82c6e5. Summary will update on new commits.
Summary by CodeRabbit
lucky.lucassantana.techdomain.