Skip to content

fix: replace print() with debugPrint() in ai_service and google_meet_service#233

Open
Satyanarayana001 wants to merge 2 commits intoAOSSIE-Org:mainfrom
Satyanarayana001:fix/replace-print-with-debugPrint
Open

fix: replace print() with debugPrint() in ai_service and google_meet_service#233
Satyanarayana001 wants to merge 2 commits intoAOSSIE-Org:mainfrom
Satyanarayana001:fix/replace-print-with-debugPrint

Conversation

@Satyanarayana001
Copy link
Copy Markdown

@Satyanarayana001 Satyanarayana001 commented Mar 20, 2026

Problem

print() calls in production code (ai_service.dart lines
552, 564, 575, 581, 583, 586, 592 and google_meet_service.dart
lines 30, 81) are flagged by flutter analyze.

Fix

Replaced all print() calls with debugPrint() which is
automatically disabled in production builds.

Summary by CodeRabbit

  • Chores
    • Replaced direct console prints with framework debug logging across core services to standardize and improve observability and troubleshooting.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 20, 2026

Walkthrough

Replaced several print() calls with debugPrint() in two Flutter service files (lib/services/ai_service.dart, lib/services/google_meet_service.dart). No control flow, return values, RPC parameters, or exception handling were changed—only the observability/logging mechanism.

Changes

Cohort / File(s) Summary
Debug logging changes
lib/services/ai_service.dart, lib/services/google_meet_service.dart
Replaced print(...) with debugPrint(...) at multiple log sites (query entry, embedding/search responses, per-meeting logs, token/error branches). No functional or API behavior modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

Dart/Flutter

Poem

🐇 I hopped through logs in moonlit code,

Swapped prints for whispers on the debug road.
No logic shifted, just softer light,
So devs can trace each quiet flight.
🥕

🚥 Pre-merge checks | ✅ 2
✅ 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 describes the main change: replacing print() with debugPrint() in the specified files, which is the core objective of this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

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.

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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@lib/services/ai_service.dart`:
- Line 8: Remove the duplicate import of 'package:flutter/foundation.dart' so
only one import statement remains; locate the repeated import line that matches
import 'package:flutter/foundation.dart' and delete the redundant instance to
satisfy Effective Dart and eliminate analyzer warnings.
🪄 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: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: e58dc05f-9f09-4fb2-a123-219bcdc0edf0

📥 Commits

Reviewing files that changed from the base of the PR and between 5afe656 and 9da67cc.

📒 Files selected for processing (2)
  • lib/services/ai_service.dart
  • lib/services/google_meet_service.dart

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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@lib/services/ai_service.dart`:
- Line 552: The debug statements in getRelevantMeetingSummaries are logging raw
user queries and meeting metadata (debugPrint calls that output the query and
meeting fields title, meeting_date, similarity, debug_info); replace those
prints with non-sensitive aggregate or redacted logs (e.g., log the number of
results, min/max/avg similarity, and a hash or redaction marker instead of the
raw query or title) and remove any logging of debug_info content that may
contain PII; update the debugPrint invocations in the
getRelevantMeetingSummaries function to only emit safe summary metrics.
🪄 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: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: bf6d0095-1c25-411f-b41f-56389c4130bf

📥 Commits

Reviewing files that changed from the base of the PR and between 9da67cc and 8053ed8.

📒 Files selected for processing (1)
  • lib/services/ai_service.dart

@Satyanarayana001
Copy link
Copy Markdown
Author

Hi 👋, just checking if any further changes are needed for this PR. Thanks!

@Satyanarayana001
Copy link
Copy Markdown
Author

Hi @SharkyBytes , just a gentle follow-up on this PR.

All requested changes have been addressed, and the PR is now ready for review. Please let me know if anything else is needed. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant