docs: align README with landing page#93
Conversation
|
Warning Review limit reached
More reviews will be available in 22 minutes and 10 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (8)
📝 WalkthroughWalkthroughDocs and landing copy rewritten to emphasize capability-first agent integration and Trust Before Invocation. Added an executable script to auto-commit ChangesMarketing and Landing Copy
Repository automation and hooks
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🚀 Preview DeployedYour changes have been deployed to a preview environment: 🌐 Landing Page: https://pr-93.preview.caplets.dev Built from commit 6855638 🤖 This comment updates automatically with each push. |
|
| Filename | Overview |
|---|---|
| README.md | Documentation-only update aligning product copy with the landing page; hero tagline, Quick Start, and Design Model sections revised. |
| apps/landing/src/pages/index.astro | Copy and label updates: renamed variables (skillifyFramework → capabilityFramework, backends → sources), updated section headings and microcopy throughout. |
| .github/workflows/pr-preview-deploy.yml | Adds path filtering, restricted PR event types, and a new Destroy preview step that tears down Cloudflare resources when a PR is closed. |
| .github/workflows/deploy.yml | Trigger changed from release:published to push:main with a paths filter; production CD now fires on every matching push without a required approval gate. |
| .github/workflows/ci.yml | Label bypass string changed from 'no changeset' to '[no changeset]', which may silently break the bypass for PRs carrying the old label. |
| alchemy.run.ts | Refactored domain variables, added www subdomain registration, exported landingPage, and replaced landingPage.url with a locally constructed URL string. |
| scripts/commit-byterover-context.ts | New post-commit helper that stages and commits .brv context changes; uses a recursion guard env var to prevent re-entrancy from the post-commit hook. |
| scripts/commit-byterover-context.test.ts | Unit tests for the three pure helper functions (checkByteRoverStatus, formatCheckWarning, buildCommitArgs); main() flow not directly covered. |
| .husky/pre-commit | Replaced full format:check + lint + typecheck with lint-staged so only staged files are checked at commit time; typecheck deferred to pre-push verify. |
| .husky/post-commit | New hook that runs commit-byterover-context.ts after every commit to auto-commit .brv context changes. |
| .lintstagedrc.json | New lint-staged config running oxfmt --check and oxlint on all staged source and config file types. |
| .changeset/config.json | Added @caplets/landing to the changeset ignore list so landing page changes don't require a changeset file. |
Reviews (7): Last reviewed commit: "ci: fix preview URL" | Re-trigger Greptile
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@README.md`:
- Line 247: The README contains a lowercase platform name "github" which should
be capitalized for consistency; replace the token "github" at the capabilities
list entry (the line containing "1. **Choose** a capability, such as `github`.")
with "GitHub" so it reads "such as `GitHub`" to match official naming and the
rest of the document.
🪄 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 Plus
Run ID: 6596450a-9f0b-4b1a-8cd9-3e9587677216
📒 Files selected for processing (1)
README.md
|
Addressed the review feedback, including the outside-summary Quick Start note: the optional Context7 backend add command is no longer in the minimal Quick Start, and the GitHub example casing is fixed. Verification: |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@scripts/commit-byterover-context.ts`:
- Line 1: The shebang in commit-byterover-context.ts is misleading because Node
cannot run TypeScript files directly; update the shebang to reference the tsx
runtime used by the hooks (e.g. change "#!/usr/bin/env node" to use "tsx") so
the script header matches how it's executed, or alternatively convert
commit-byterover-context.ts to a compiled JavaScript file and keep the original
Node shebang; ensure the runtime indicated in the shebang aligns with the script
format so execution via pnpm exec tsx remains correct.
🪄 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 Plus
Run ID: fe28fd1f-f51f-490d-879c-0900ce1c50f3
⛔ Files ignored due to path filters (23)
.brv/context-tree/design/caplets/caplets_module_findings.mdis excluded by!.brv/**.brv/context-tree/design/caplets/formatting_check.mdis excluded by!.brv/**.brv/context-tree/design/caplets/hero_copy.mdis excluded by!.brv/**.brv/context-tree/design/caplets/landing_positioning.mdis excluded by!.brv/**.brv/context-tree/design/caplets/quick_start.mdis excluded by!.brv/**.brv/context-tree/design/caplets/readme_header_badges.mdis excluded by!.brv/**.brv/context-tree/design/caplets/readme_md.mdis excluded by!.brv/**.brv/context-tree/design/caplets/readme_section.mdis excluded by!.brv/**.brv/context-tree/facts/ci/ci.mdis excluded by!.brv/**.brv/context-tree/facts/extracted_facts/extracted_facts.mdis excluded by!.brv/**.brv/context-tree/facts/mcp_example/mcp_example.mdis excluded by!.brv/**.brv/context-tree/facts/minimal_quick_start/minimal_quick_start.mdis excluded by!.brv/**.brv/context-tree/facts/pnpm_format_check/pnpm_format_check.mdis excluded by!.brv/**.brv/context-tree/facts/pnpm_verify/pnpm_verify.mdis excluded by!.brv/**.brv/context-tree/facts/pr_93/pr_93.mdis excluded by!.brv/**.brv/context-tree/facts/project/179803b.mdis excluded by!.brv/**.brv/context-tree/facts/project/ab9efe5.mdis excluded by!.brv/**.brv/context-tree/facts/project/extracted_facts.mdis excluded by!.brv/**.brv/context-tree/facts/project/post_commit_hook_brv.mdis excluded by!.brv/**.brv/context-tree/facts/project/uncategorized.mdis excluded by!.brv/**.brv/context-tree/facts/review_threads/review_threads.mdis excluded by!.brv/**.brv/context-tree/facts/staged_path_example/staged_path_example.mdis excluded by!.brv/**.brv/context-tree/src/caplets/caplets_module_findings.mdis excluded by!.brv/**
📒 Files selected for processing (9)
.changeset/config.json.github/workflows/changeset-reminder.yml.github/workflows/ci.yml.husky/post-commit.husky/pre-pushREADME.mdapps/landing/src/pages/index.astroscripts/commit-byterover-context.test.tsscripts/commit-byterover-context.ts
💤 Files with no reviewable changes (1)
- .github/workflows/changeset-reminder.yml
✅ Files skipped from review due to trivial changes (3)
- .husky/post-commit
- .changeset/config.json
- README.md
| on: | ||
| release: | ||
| types: [published] | ||
| push: | ||
| branches: | ||
| - main | ||
| paths: | ||
| - .github/workflows/deploy.yml | ||
| - alchemy.run.ts | ||
| - apps/** | ||
| - package.json | ||
| - pnpm-lock.yaml | ||
| - pnpm-workspace.yaml | ||
| - scripts/alchemy-*.ts | ||
| workflow_dispatch: |
There was a problem hiding this comment.
main
The trigger was changed from release: published (gated) to push: main with a paths filter, with no environment: protection gate in the job. This PR itself touches alchemy.run.ts and apps/**, so merging it will immediately fire a production deployment. Any future commit touching those paths will do the same. If the intent is continuous deployment, an environment: production protection rule is the standard mechanism to require approval before prod deploys run.
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
1 participant
Summary
Test Plan
Summary by CodeRabbit