Skip to content

Conversation

@anish-work
Copy link
Contributor

@anish-work anish-work commented Nov 12, 2025

Q/A checklist

  • I have tested my UI changes on mobile and they look acceptable
  • I have tested changes to the workflows in both the API and the UI
  • I have done a code review of my changes and looked at each line of the diff + the references of each function I have changed
  • My changes have not increased the import time of the server
How to check import time?

time python -c 'import server'

You can visualize this using tuna:

python3 -X importtime -c 'import server' 2> out.log && tuna out.log

To measure import time for a specific library:

$ time python -c 'import pandas'

________________________________________________________
Executed in    1.15 secs    fish           external
   usr time    2.22 secs   86.00 micros    2.22 secs
   sys time    0.72 secs  613.00 micros    0.72 secs

To reduce import times, import libraries that take a long time inside the functions that use them instead of at the top of the file:

def my_function():
    import pandas as pd
    ...

Legal Boilerplate

Look, I get it. The entity doing business as “Gooey.AI” and/or “Dara.network” was incorporated in the State of Delaware in 2020 as Dara Network Inc. and is gonna need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Dara Network Inc can use, modify, copy, and redistribute my contributions, under its choice of terms.

@coderabbitai
Copy link

coderabbitai bot commented Nov 12, 2025

📝 Walkthrough

Walkthrough

This pull request updates the templates/functions/called_functions.html template by adding a .tool-name CSS rule (font-size: 14px) to the existing style block and applying the tool-name class to the <strong> element in the summary line. The change is cosmetic and does not alter logic, data flow, or public interfaces.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Small, localized HTML/CSS change; review focus: correct selector and intended element class usage.

Suggested reviewers

  • devxpy

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: reducing font size for the tool name in the HTML template.
Description check ✅ Passed The description includes all required sections from the template with the Q/A checklist completed and legal boilerplate retained.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch widget_tool_call_fixes

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6b7ad28 and 30261bc.

📒 Files selected for processing (1)
  • templates/functions/called_functions.html (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: test (3.10.12, 1.8.3)
🔇 Additional comments (2)
templates/functions/called_functions.html (2)

32-34: CSS rule addition looks good.

The .tool-name class is syntactically correct and appropriately named to reflect its purpose. The font-size value aligns with the PR objective to reduce the tool name font size.


39-42: Class application is correct.

The tool-name class is properly applied to the <strong> element wrapping the tool title, and the template syntax remains intact. The change cleanly applies the new styling without affecting surrounding content or Jinja2 template logic.


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

@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: 0

🧹 Nitpick comments (1)
templates/functions/called_functions.html (1)

32-34: Consider using a more specific selector to target only the tool name.

The CSS rule applies to all <strong> elements in this template, including both the tool name (line 39) and section headers like "Inputs" and "Return value" (lines 48, 55). Given the PR title specifies "reduce font size for tool name" (singular), this broad selector may be unintended.

If only the tool name should be affected, add a class to the tool name's <strong> element and target it explicitly:

 <summary><strong>
+<strong class="tool-name">
 {% if item.is_running %}<div class='gooey-spinner'></div>&nbsp;{% else %}🛠️{% endif %} 
 {{ item.title.replace("_", "<wbr>_") | safe }}
-</strong></summary>
+</strong></strong></summary>

Then update the CSS:

-strong {
+.tool-name {
     font-size: 14px;
 }

If reducing font size uniformly for all <strong> elements is intentional, clarify the PR title accordingly.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0643b1a and 6b7ad28.

📒 Files selected for processing (1)
  • templates/functions/called_functions.html (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: test (3.10.12, 1.8.3)

@anish-work anish-work force-pushed the widget_tool_call_fixes branch from 6b7ad28 to 30261bc Compare November 12, 2025 07:21
@anish-work anish-work requested a review from devxpy November 12, 2025 07:21
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.

2 participants