fix: cleanup multiline DemoSource

[javier-godoy]

Close #85

Summary by CodeRabbit

• Bug Fixes
  • Improved handling of code demo blocks to ensure proper display and filtering.

[coderabbitai]

Walkthrough

The cleanupCode function in code-viewer.ts is updated to remove complete multiline @DemoSource blocks using parenthesis-aware preprocessing, replacing the previous per-line filtering approach.

Changes

Cohort / File(s)	Summary
Code cleanup logic enhancement base/src/main/resources/META-INF/resources/frontend/code-viewer.ts	Updated cleanupCode to preprocess multiline @DemoSource(...) blocks by nulling lines while tracking parenthesis balance, then removed the explicit @DemoSource exclusion from the final filter.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: cleanup multiline DemoSource' clearly and concisely describes the main change: implementing proper handling for multiline @DemoSource annotations in the cleanupCode function.
Linked Issues check	✅ Passed	The code changes directly address issue #85 by implementing parenthesis-aware block removal for multiline @DemoSource constructs, ensuring all lines of the annotation are properly removed instead of just the first line.
Out of Scope Changes check	✅ Passed	All changes are scoped to the cleanupCode function in code-viewer.ts and directly address the multiline @DemoSource handling objective from issue #85, with no unrelated modifications present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-85

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@base/src/main/resources/META-INF/resources/frontend/code-viewer.ts`:
- Around line 328-335: The `@DemoSource` removal code only looks for '(' on the
same line; instead compute a parenthesis balance for the annotation start (count
'(' minus ')' on the current line) and if balance > 0 iterate forward updating
balance with counts of '(' and ')' on each subsequent line, setting each visited
lines[k] = null until balance <= 0 or end of file; if balance == 0 just null the
single line. Update the block that inspects lines[i] (the `@DemoSource` handling
loop using variables lines and i) to use this parenthesis-balance loop so
wrapped annotations spanning multiple lines are fully removed.

🪄 Autofix (Beta)

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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1969207e-ef04-4ea2-b823-ad672ab7e6ce

📥 Commits

Reviewing files that changed from the base of the PR and between fcf590d and 68a1f00.

📒 Files selected for processing (1)

• base/src/main/resources/META-INF/resources/frontend/code-viewer.ts