Skip to content

Fix: Next.js 16#10713

Merged
Meldiron merged 1 commit into1.8.xfrom
fix-next-16
Oct 27, 2025
Merged

Fix: Next.js 16#10713
Meldiron merged 1 commit into1.8.xfrom
fix-next-16

Conversation

@Meldiron
Copy link
Copy Markdown
Contributor

@Meldiron Meldiron commented Oct 27, 2025 •
edited by stnguyen90
Loading

What does this PR do?

Fixes SSR detection for Next.js.

Also upgrades Console to latest.

Test Plan

Manual QA

CleanShot 2025-10-27 at 17 04 32@2x

Related PRs and Issues

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Oct 27, 2025 •
edited
Loading

📝 Walkthrough

Walkthrough

This pull request updates the Docker image tag for the appwrite-console service in docker-compose.yml from version 7.4.7 to 7.4.11. The change is a single-line version bump with no modifications to service structure, environment variables, ports, or any other configuration settings.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Update console #9792: Updates the Docker image tag for the appwrite-console service in docker-compose configuration files.
  • Update console #9798: Modifies the Docker image tag for the appwrite-console service in docker-compose configuration.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "Fix: Next.js 16" is related to the pull request's objectives, which include fixing SSR detection for Next.js. According to the PR description and objectives, the main changes involve fixing server-side rendering (SSR) detection for Next.js and upgrading the Console to the latest version. The title appropriately references the Next.js fix as a core part of the changeset, even though it doesn't explicitly mention SSR or the secondary console upgrade. The title is concise and specifically identifies the primary fix, rather than being overly generic or off-topic.
Description Check ✅ Passed The pull request description is clearly related to the changeset. It articulates the main objectives—fixing SSR detection for Next.js and upgrading Console to the latest version—which are confirmed by the raw summary showing the appwrite-console service image tag updated from 7.4.7 to 7.4.11. The description also includes a test plan (manual QA with a screenshot), demonstrates adherence to contributing guidelines, and provides sufficient context for understanding the changes. The description is directly relevant to the code modifications.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-next-16

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 27, 2025

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
binutils 2.44-r2 CVE-2025-5244 HIGH
binutils 2.44-r2 CVE-2025-5245 HIGH
libxml2 2.13.8-r0 CVE-2025-49794 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49796 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49795 HIGH
libxml2 2.13.8-r0 CVE-2025-6021 HIGH
pcre2 10.43-r1 CVE-2025-58050 CRITICAL
golang.org/x/crypto v0.31.0 CVE-2025-22869 HIGH
golang.org/x/oauth2 v0.24.0 CVE-2025-22868 HIGH
stdlib 1.22.10 CVE-2025-47907 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

coderabbitai[bot]
coderabbitai bot reviewed Oct 27, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7047ffe and 7134938.

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • docker-compose.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: scan

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 27, 2025

✨ Benchmark results

  • Requests per second: 1,187
  • Requests with 200 status code: 213,668
  • P99 latency: 0.165839353

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,187 1,150
200 213,668 207,008
P99 0.165839353 0.180795755

@Meldiron Meldiron merged commit dde6968 into 1.8.x Oct 27, 2025
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@loks0n loks0n loks0n approved these changes

+1 more reviewer

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Meldiron @loks0n