Resolve CodeQL trivial-conditional in DecoderText startup path

[Copilot]

This PR addresses the repository’s flagged code scanning alert (#6) by removing a trivially true conditional in DecoderText. The change is scoped to the alert location and preserves existing behavior.

• Alert remediation

  • Removed an unused local variable (animation) that was never assigned.
  • Simplified the startup guard to eliminate the always-true negation path reported by CodeQL (js/trivial-conditional).

• Code impact

  • File updated: app/components/decoder-text/decoder-text.jsx
  • No functional logic changes beyond removing dead conditional state.

// before
let animation;
if (start && !animation && !reduceMotion) {
  startSpring();
}

// after
if (start && !reduceMotion) {
  startSpring();
}

[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:49am UTC

[cloudflare-workers-and-pages]

Deploying zrl-dev with    Cloudflare Pages

Latest commit:	2fdd13b
Status:	✅  Deploy successful!
Preview URL:	https://e636efb9.zrl-dev.pages.dev
Branch Preview URL:	https://copilot-fix-code-scanning-al.zrl-dev.pages.dev

View logs

[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 (4826596)	1312	1311	99.92%
Head commit (2fdd13b)	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 (#79)	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.}

[Copilot]

🟢 Approval recommended

The change removes an unused variable and an always-true conditional branch without altering the component’s effective behavior.

Pull request overview

Remediates a CodeQL js/trivial-conditional finding in DecoderText by removing a dead local variable and simplifying the effect’s startup guard while preserving existing runtime behavior.

Changes:

• Removed an unused animation local that was never assigned.
• Simplified the startup conditional from start && !animation && !reduceMotion to start && !reduceMotion.

File summaries

File	Description
app/components/decoder-text/decoder-text.jsx	Removes dead state and simplifies the startup guard to address the CodeQL trivial-conditional alert.

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.