Repository navigation
fix(web): link release notes to GitHub - #2114
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📜 Recent review details⏰ Context from checks skipped due to timeout. (3)
🧰 Additional context used📓 Path-based instructions (4)**/*.{ts,tsx}📄 CodeRabbit inference engine (AGENTS.md)
Files:
web/**/*.{ts,tsx}📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*📄 CodeRabbit inference engine (AGENTS.md)
Files:
⚙️ CodeRabbit configuration file
Files:
web/**⚙️ CodeRabbit configuration file
Files:
🧠 Learnings (10)📓 Common learnings📚 Learning: 2026-08-07T01:57:16.417ZApplied to files:
📚 Learning: 2026-08-07T01:57:07.096ZApplied to files:
📚 Learning: 2026-08-07T01:57:07.096ZApplied to files:
📚 Learning: 2026-08-07T01:57:16.417ZApplied to files:
📚 Learning: 2026-08-07T01:57:16.417ZApplied to files:
📚 Learning: 2026-06-17T03:03:34.545ZApplied to files:
📚 Learning: 2026-08-07T01:57:16.417ZApplied to files:
📚 Learning: 2026-08-07T01:57:16.417ZApplied to files:
📚 Learning: 2026-08-07T01:57:16.417ZApplied to files:
🔇 Additional comments (2)
📝 WalkthroughWalkthroughThe web site removes local changelog and npm freshness validation. Release navigation now targets GitHub Releases, the legacy changelog route redirects there, version displays are removed, and contributor guidance is updated. ChangesGitHub Releases migration
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![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 `@web/src/data/releases.ts`:
- Around line 24-35: Update the PR validation report associated with the release
entry to explicitly list the focused web commands bun run web:typecheck and bun
run web:build, and run both checks before reporting them.
🪄 Autofix
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: ASSERTIVE
Plan: Pro
Run ID: f8c58d4e-441e-4721-be6f-b10acf2c0bee
📒 Files selected for processing (1)
web/src/data/releases.ts
📜 Review details
⏰ Context from checks skipped due to timeout. (3)
- GitHub Check: smoke-and-tests (24.11.x)
- GitHub Check: smoke-and-tests (22)
- GitHub Check: typecheck
🧰 Additional context used
📓 Path-based instructions (8)
**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
Use TypeScript strict mode and ESM imports throughout the source code.
Run
bun run typecheckandbun run typecheck:type-testsfor TypeScript changes when applicable.
Files:
web/src/data/releases.ts
**/*.{tsx,ts}
📄 CodeRabbit inference engine (AGENTS.md)
Use React and Ink patterns for terminal UI components.
Files:
web/src/data/releases.ts
web/**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
When changing the web application, run the web typecheck and build checks.
Files:
web/src/data/releases.ts
web/src/data/releases.ts
📄 CodeRabbit inference engine (AGENTS.md)
Do not edit
web/src/data/releases.tsin ordinary feature or bugfix PRs; it is owned by the release/web process.Do not edit
web/src/data/releases.tsduring ordinary feature or bug-fix work; only modify it in an explicit release/web changelog PR.
Files:
web/src/data/releases.ts
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (AGENTS.md)
Do not add new Python code, Python provider paths, or Python dependencies without explicit maintainer approval.
**/*.{ts,tsx,js,jsx}: Follow the existing code style in touched source files, prefer small readable changes, avoid unrelated reformatting, and keep comments useful and concise.
Preserve existing repository patterns unless intentionally refactoring them, and avoid broad rewrites or unnecessary generated changes.
Review AI-assisted code for correctness, style consistency, unnecessary changes, and adherence to project architecture before submitting it.
Files:
web/src/data/releases.ts
**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Update documentation when setup, commands, or user-facing behavior changes.
Files:
web/src/data/releases.ts
⚙️ CodeRabbit configuration file
**/*: Apply the OpenClaude maintainer review rubric from AGENTS.md. Review the current diff, not stale discussion context. Separate real blockers from suggestions. Do not request changes for vague style churn. Treat approval as merge-ready from CodeRabbit's side, pending required human review and GitHub Checks. If checks are failing or unavailable, say so clearly instead of implying the PR is fully ready.
Files:
web/src/data/releases.ts
web/**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
When changing files under
web/, runbun run web:typecheckandbun run web:build.
Files:
web/src/data/releases.ts
web/**
⚙️ CodeRabbit configuration file
web/**: Review browser extension changes for content-script isolation, message validation, cross-origin assumptions, permission surfaces, and failures that could leak prompts or credentials.
Files:
web/src/data/releases.ts
4b62ca9 to
c248098
Compare
jatmn
changed the title
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 @.github/workflows/web-release-freshness.yml:
- Around line 15-16: Update the actions/checkout step in the web release
freshness workflow to set persist-credentials to false, ensuring later Bun steps
cannot access persisted checkout credentials; leave the existing checkout action
and other options unchanged.
In `@web/scripts/verify-dist.test.ts`:
- Line 175: Extend the verification tests around the existing “release/web
process” assertion to cover the CHECK_NPM_FRESHNESS gate, including its enabled
and disabled behavior and expected failure handling.
🪄 Autofix
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: ASSERTIVE
Plan: Pro
Run ID: a851396f-92bf-4848-b4bf-812ffb3609cc
📒 Files selected for processing (4)
.github/workflows/web-release-freshness.ymlweb/scripts/verify-dist.test.tsweb/scripts/verify-dist.tsweb/src/data/releases.ts
📜 Review details
⏰ Context from checks skipped due to timeout. (3)
- GitHub Check: smoke-and-tests (22)
- GitHub Check: smoke-and-tests (24.11.x)
- GitHub Check: typecheck
🧰 Additional context used
📓 Path-based instructions (11)
**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
Use TypeScript strict mode and ESM imports throughout the source code.
Run
bun run typecheckandbun run typecheck:type-testsfor TypeScript changes when applicable.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
**/*.{tsx,ts}
📄 CodeRabbit inference engine (AGENTS.md)
Use React and Ink patterns for terminal UI components.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
web/**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
When changing the web application, run the web typecheck and build checks.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (AGENTS.md)
Do not add new Python code, Python provider paths, or Python dependencies without explicit maintainer approval.
**/*.{ts,tsx,js,jsx}: Follow the existing code style in touched source files, prefer small readable changes, avoid unrelated reformatting, and keep comments useful and concise.
Preserve existing repository patterns unless intentionally refactoring them, and avoid broad rewrites or unnecessary generated changes.
Review AI-assisted code for correctness, style consistency, unnecessary changes, and adherence to project architecture before submitting it.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Update documentation when setup, commands, or user-facing behavior changes.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
⚙️ CodeRabbit configuration file
**/*: Apply the OpenClaude maintainer review rubric from AGENTS.md. Review the current diff, not stale discussion context. Separate real blockers from suggestions. Do not request changes for vague style churn. Treat approval as merge-ready from CodeRabbit's side, pending required human review and GitHub Checks. If checks are failing or unavailable, say so clearly instead of implying the PR is fully ready.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
web/**/*
📄 CodeRabbit inference engine (CONTRIBUTING.md)
When changing files under
web/, runbun run web:typecheckandbun run web:build.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
web/**
⚙️ CodeRabbit configuration file
web/**: Review browser extension changes for content-script isolation, message validation, cross-origin assumptions, permission surfaces, and failures that could leak prompts or credentials.
Files:
web/scripts/verify-dist.tsweb/src/data/releases.tsweb/scripts/verify-dist.test.ts
.github/**
⚙️ CodeRabbit configuration file
.github/**: Review CI and release workflow changes for token permissions, third-party actions, pull_request_target usage, artifact upload/download behavior, shell injection, and whether checks still run on the actual PR head. Block on broadened write permissions or unpinned/excessively trusted external execution unless clearly justified.
Files:
.github/workflows/web-release-freshness.yml
web/src/data/releases.ts
📄 CodeRabbit inference engine (AGENTS.md)
Do not edit
web/src/data/releases.tsin ordinary feature or bugfix PRs; it is owned by the release/web process.Do not edit
web/src/data/releases.tsduring ordinary feature or bug-fix work; only modify it in an explicit release/web changelog PR.
Files:
web/src/data/releases.ts
**/*.{test,spec}.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
Add or update tests when behavior changes, and run the narrowest useful focused test checks.
Files:
web/scripts/verify-dist.test.ts
**/*.{test,spec}.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
**/*.{test,spec}.{ts,tsx,js,jsx}: Add or update tests when a code change affects behavior.
Use focused tests such asbun test ./path/to/test-file.test.tswhen validating a narrowly scoped change.
Files:
web/scripts/verify-dist.test.ts
🪛 zizmor (1.29.0)
.github/workflows/web-release-freshness.yml
[warning] 15-16: credential persistence through GitHub Actions artifacts (artipacked): does not set persist-credentials: false
(artipacked)
[info] 12-12: workflow or action definition without a name (anonymous-definition): this job
(anonymous-definition)
[warning] 3-6: insufficient job-level concurrency limits (concurrency-limits): workflow is missing concurrency setting
(concurrency-limits)
🔇 Additional comments (2)
web/scripts/verify-dist.ts (1)
98-99: LGTM!Also applies to: 113-122
web/src/data/releases.ts (1)
143-144: LGTM!
c248098 to
78038c3
Compare
jatmn
changed the title
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 `@web/scripts/verify-dist.ts`:
- Around line 46-51: Update verifyDist in web/scripts/verify-dist.ts at lines
46-51 to require target="_blank" and rel="noopener" on both release links, and
require noindex on the legacy changelog redirect output. Update the valid
fixture in web/scripts/verify-dist.test.ts at lines 20-30 with these attributes,
then add mutation cases removing each attribute and assert verification fails.
🪄 Autofix
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: ASSERTIVE
Plan: Pro
Run ID: 2f1a15ae-67a1-41e0-bcf8-4a256bcb08b4
📒 Files selected for processing (14)
AGENTS.mdCONTRIBUTING.mdweb/astro.config.mjsweb/scripts/verify-dist.test.tsweb/scripts/verify-dist.tsweb/src/components/DocsSidebar.astroweb/src/components/Footer.astroweb/src/components/Nav.astroweb/src/data/docsNav.tsweb/src/data/releases.tsweb/src/data/site.tsweb/src/pages/changelog.astroweb/src/pages/docs/index.astroweb/src/pages/index.astro
💤 Files with no reviewable changes (2)
- web/src/pages/changelog.astro
- web/src/data/releases.ts
📜 Review details
🧰 Additional context used
📓 Path-based instructions (6)
web/**/*
📄 CodeRabbit inference engine (AGENTS.md)
Run web typecheck and build validation when changing files under
web/.
Files:
web/src/components/DocsSidebar.astroweb/astro.config.mjsweb/src/pages/docs/index.astroweb/src/components/Nav.astroweb/src/pages/index.astroweb/src/data/site.tsweb/src/data/docsNav.tsweb/src/components/Footer.astroweb/scripts/verify-dist.tsweb/scripts/verify-dist.test.ts
**/*
📄 CodeRabbit inference engine (AGENTS.md)
**/*: Run the narrowest useful validation checks for each change and list the exact commands in the PR.
Do not add a manually maintained release-notes data source to the static site; link to GitHub Releases instead.
**/*: Keep pull requests focused on one problem or feature, and avoid mixing unrelated cleanup or refactors.
Preserve existing repository patterns unless intentionally refactoring them.
Update documentation when setup, commands, or user-facing behavior changes.
Do not add manually maintained release-note data to the static site; link release-notes navigation to GitHub Releases.
Review AI-generated changes for correctness, style consistency, unnecessary changes, and adherence to project architecture before submitting them.
Follow the existing code style in touched files.
Prefer small, readable changes over broad rewrites.
Do not reformat unrelated files.
Keep comments useful and concise.
Provider changes must avoid breaking third-party providers while fixing first-party behavior.
When changing provider behavior, test the exact provider/model path changed when possible and document affected providers, limitations, and follow-up work.
Do not assign or use provider tags; provider tags are controlled by maintainers.
Run the narrowest meaningful validation command for the touched area, and do not merge changes that fail CI checks.
Files:
web/src/components/DocsSidebar.astroweb/astro.config.mjsweb/src/pages/docs/index.astroAGENTS.mdweb/src/components/Nav.astroweb/src/pages/index.astroweb/src/data/site.tsweb/src/data/docsNav.tsweb/src/components/Footer.astroweb/scripts/verify-dist.tsweb/scripts/verify-dist.test.tsCONTRIBUTING.md
⚙️ CodeRabbit configuration file
**/*: Apply the OpenClaude maintainer review rubric from AGENTS.md. Review the current diff, not stale discussion context. Separate real blockers from suggestions. Do not request changes for vague style churn. Treat approval as merge-ready from CodeRabbit's side, pending required human review and GitHub Checks. If checks are failing or unavailable, say so clearly instead of implying the PR is fully ready.
Files:
web/src/components/DocsSidebar.astroweb/astro.config.mjsweb/src/pages/docs/index.astroAGENTS.mdweb/src/components/Nav.astroweb/src/pages/index.astroweb/src/data/site.tsweb/src/data/docsNav.tsweb/src/components/Footer.astroweb/scripts/verify-dist.tsweb/scripts/verify-dist.test.tsCONTRIBUTING.md
web/**
⚙️ CodeRabbit configuration file
web/**: Review browser extension changes for content-script isolation, message validation, cross-origin assumptions, permission surfaces, and failures that could leak prompts or credentials.
Files:
web/src/components/DocsSidebar.astroweb/astro.config.mjsweb/src/pages/docs/index.astroweb/src/components/Nav.astroweb/src/pages/index.astroweb/src/data/site.tsweb/src/data/docsNav.tsweb/src/components/Footer.astroweb/scripts/verify-dist.tsweb/scripts/verify-dist.test.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
**/*.{ts,tsx}: Use TypeScript with strict mode and ESM imports.
Prefer existing service, provider, settings, permission, and UI patterns over introducing new abstractions.
Files:
web/src/data/site.tsweb/src/data/docsNav.tsweb/scripts/verify-dist.tsweb/scripts/verify-dist.test.ts
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Add or update tests when a code change affects behavior.
Files:
web/src/data/site.tsweb/src/data/docsNav.tsweb/scripts/verify-dist.tsweb/scripts/verify-dist.test.ts
{README.md,CONTRIBUTING.md,docs/**,.github/pull_request_template.md}
⚙️ CodeRabbit configuration file
{README.md,CONTRIBUTING.md,docs/**,.github/pull_request_template.md}: Review docs for accuracy against current code behavior. Flag security or provider claims that overpromise, stale install commands, missing setup caveats, and instructions that could push users toward unsafe credential handling. Keep purely wording-level suggestions non-blocking.
Files:
CONTRIBUTING.md
🧠 Learnings (1)
📓 Common learnings
Learnt from: CR
Repo: Gitlawb/openclaude
Timestamp: 2026-08-11T23:12:52.725Z
Learning: Keep changes focused on one problem and avoid unrelated formatting, renames, dependency changes, or broad rewrites.
Learnt from: CR
Repo: Gitlawb/openclaude
Timestamp: 2026-08-11T23:12:52.725Z
Learning: Add or update tests when behavior changes.
Learnt from: CR
Repo: Gitlawb/openclaude
Timestamp: 2026-08-11T23:12:52.725Z
Learning: Update documentation when setup, commands, provider behavior, or user-facing behavior changes.
Learnt from: CR
Repo: Gitlawb/openclaude
Timestamp: 2026-08-11T23:12:52.725Z
Learning: Do not change the Node.js runtime or Bun development workflow without prior maintainer agreement.
Learnt from: CR
Repo: Gitlawb/openclaude
Timestamp: 2026-08-11T23:12:52.725Z
Learning: Do not skip tests for behavior changes.
Learnt from: CR
Repo: Gitlawb/openclaude
Timestamp: 2026-08-11T23:12:52.725Z
Learning: Address CodeRabbit or maintainer feedback before requesting more review.
🪛 LanguageTool
AGENTS.md
[style] ~104-~104: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ess it before requesting more review. - Do not add a manually maintained release-n...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
🔇 Additional comments (10)
AGENTS.md (1)
77-77: LGTM!Also applies to: 104-104
CONTRIBUTING.md (1)
136-136: LGTM!Also applies to: 154-154
web/src/data/site.ts (1)
10-10: LGTM!web/src/data/docsNav.ts (1)
1-6: LGTM!Also applies to: 37-43
web/src/components/DocsSidebar.astro (1)
15-15: LGTM!web/src/components/Footer.astro (1)
27-27: LGTM!Also applies to: 57-57, 67-68
web/src/components/Nav.astro (1)
8-8: LGTM!Also applies to: 21-28, 39-39
web/src/pages/docs/index.astro (1)
41-41: LGTM!web/src/pages/index.astro (1)
104-104: LGTM!web/astro.config.mjs (1)
7-9: LGTM!
jatmn
left a comment
jatmn
left a comment
There was a problem hiding this comment.
@kevincodex1 LGTM
Will make the site more basic for now when it comes to release notes.. but also prevents alot of slop in the notes as well as the current ci issues on main
|
will redeploy the site in a bit |
2 participants
Summary
/changelog/URLs with a noindex redirect to GitHub ReleasesRoot cause
The static changelog duplicated release data, made the displayed version stale,
and added an unrelated npm-registry dependency to web builds. GitHub Releases
is the canonical release-notes source.
Validation
bun test web/scripts/verify-dist.test.tsbun run --cwd web typecheckbun run --cwd web buildSummary by CodeRabbit
New Features
Updates
Documentation