Resolve CodeQL trivial conditional in image video playback effect

[Copilot]

Addresses code scanning alert #7 by removing a redundant conditional that CodeQL flagged as always true in the image component’s video playback effect. The change is intentionally minimal and limited to the alerted logic path.

• Root cause

  • In ImageElements (app/components/image/image.jsx), the else if branch re-checked inViewport immediately after if (!inViewport), making that sub-condition trivially true.

• Change made

  • Simplified the playback guard by removing the redundant inViewport check while preserving the existing behavior gates (reduceMotion, play).

• Code snippet

  if (!inViewport) {
    pauseVideo();
  } else if (!reduceMotion && play) {
    playVideo();
  }

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
zrl.dev	Ready	Preview	Sep 2, 2026 3:50am UTC

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric	Results
Duplication	0

View in Codacy

🟢 Coverage ∅ diff coverage · +0.00% coverage variation

Metric	Results
Coverage variation	✅ +0.00% coverage variation (-1.00%)
Diff coverage	✅ ∅ diff coverage

View coverage diff in Codacy

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (b5b86d7)	1312	1311	99.92%
Head commit (862039a)	1312 (+0)	1311 (+0)	99.92% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#78)	0	0	∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}

[cloudflare-workers-and-pages]

Deploying zrl-dev with    Cloudflare Pages

Latest commit:	862039a
Status:	✅  Deploy successful!
Preview URL:	https://b1783a90.zrl-dev.pages.dev
Branch Preview URL:	https://copilot-fix-code-scanning-al-vzrn.zrl-dev.pages.dev

View logs

[Copilot]

🟢 Approval recommended

The change is a minimal, behavior-preserving simplification that resolves the trivially-true conditional without introducing new logic paths.

Pull request overview

This PR addresses a CodeQL code scanning finding in the ImageElements component by simplifying a trivially-true conditional in the video playback useEffect, while preserving the existing gating behavior (inViewport, reduceMotion, play, and the early returns/pausing logic).

Changes:

• Removes the redundant inViewport check from the else if branch following if (!inViewport).
• Keeps the same effective playback behavior by relying on the else branch to imply inViewport is truthy.

File summaries

File	Description
app/components/image/image.jsx	Simplifies the video playback guard to remove a redundant viewport sub-condition flagged by CodeQL.

Review details

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.