Fix failing test and improve parseJSONFromAI

[APPLEPIE6969]

I noticed that the tests were failing due to an incorrect import in lib/ai.test.ts. I fixed the import to point to lib/ai-utils.ts. Additionally, one of the tests assert.throws(() => parseJSONFromAI("")) was failing because the parseJSONFromAI function didn't explicitly throw an error on empty strings (it was replacing strings and then falling back to JSON.parse(""), which threw an error, but it's cleaner to handle empty strings explicitly). I added a guard to parseJSONFromAI to throw a SyntaxError on empty strings. All tests now pass.

---

PR created automatically by Jules for task 6713942556894457895 started by @APPLEPIE6969

Summary by CodeRabbit

• Bug Fixes
  • Improved error handling for JSON parsing by adding upfront validation that throws clear errors for null, empty, or whitespace-only inputs.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
studyflow	Ready	Preview, Comment	Mar 16, 2026 1:18pm

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 271792c5-f9ff-4094-bec6-2c9d16c44192

📥 Commits

Reviewing files that changed from the base of the PR and between 355b8d9 and 1c164c4.

📒 Files selected for processing (2)

• lib/ai-utils.ts
• lib/ai.test.ts

---

📝 Walkthrough

Walkthrough

The pull request adds input validation to the parseJSONFromAI function to throw a SyntaxError when receiving null, empty, or whitespace-only input, and updates the corresponding test file import to reference the new location of this function.

Changes

Cohort / File(s)	Summary
Input Validation Enhancement lib/ai-utils.ts	Added upfront null/empty/whitespace validation in parseJSONFromAI that throws SyntaxError before attempting JSON parsing; existing fallback logic preserved for valid input.
Import Path Update lib/ai.test.ts	Updated import statement to reference parseJSONFromAI from ai-utils.ts instead of ai.ts; test cases remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• Refactor: proactive code fixes and improvements #60: Directly relates to the parseJSONFromAI function being modified; changes involve the same utility function for JSON parsing from AI output.

Poem

🐰 A watchful bunny checks the gates,
Ensuring input's sound before it parses,
No empty holes to lead to fates,
Just clean validation—the bunny traverses! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately captures the two main changes: fixing a failing test by correcting an import path and improving parseJSONFromAI with input validation.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/test-and-json-parse-6713942556894457895

📝 Coding Plan

• Generate coding plan for human review comments

---

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

Tip

CodeRabbit can use your project's `biome` configuration to improve the quality of JS/TS/CSS/JSON code reviews.

Add a configuration file to your project to customize how CodeRabbit runs biome.