Skip to content

Conversation

@hyacinthus
Copy link
Collaborator

This PR includes:

  • Frontend skill box display in chat interface
  • Local frontend development improvements
  • GPT image model updated to version 1.5
  • Docker compose configuration fixes
  • Debug authentication removed
  • Various lint fixes
  • Dependency updates

Copilot AI review requested due to automatic review settings December 27, 2025 12:16
@hyacinthus hyacinthus merged commit 8026f0f into main Dec 27, 2025
5 checks passed
@hyacinthus hyacinthus deleted the hyacinthus branch December 27, 2025 12:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 21 out of 24 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (9)

app/admin/api.py:77

  • This comment appears to contain commented-out code.
    # if not input.owner:
    #     raise IntentKitAPIError(
    #         status_code=400, key="BadRequest", message="Owner is required"

app/admin/api.py:85

  • This comment appears to contain commented-out code.
    # if user_id:
    #     if input.owner != user_id:
    #         raise IntentKitAPIError(
    #             status_code=400,
    #             key="BadRequest",
    #             message="Owner does not match user ID",

app/admin/api.py:92

  • This comment appears to contain commented-out code.
    # if user:
    #     max_fee += user.nft_count * 10
    # if input.fee_percentage and input.fee_percentage > max_fee:
    #     raise IntentKitAPIError(
    #         status_code=400, key="BadRequest", message="Fee percentage too high"

app/admin/api.py:127

  • This comment appears to contain commented-out code.
    # if not input.owner:
    #     raise IntentKitAPIError(
    #         status_code=400, key="BadRequest", message="Owner is required"

app/admin/api.py:135

  • This comment appears to contain commented-out code.
    # if user_id:
    #     if input.owner != user_id:
    #         raise IntentKitAPIError(
    #             status_code=400,
    #             key="BadRequest",
    #             message="Owner does not match user ID",

app/admin/api.py:142

  • This comment appears to contain commented-out code.
    #     if user:
    #         max_fee += user.nft_count * 10
    # if input.fee_percentage and input.fee_percentage > max_fee:
    #     raise IntentKitAPIError(
    #         status_code=400, key="BadRequest", message="Fee percentage too high"

app/admin/credit.py:122

  • This comment appears to contain commented-out code.
# class AdjustmentRequest(BaseModel):
#     """Request model for adjusting a user account."""

#     upstream_tx_id: Annotated[
#         str, Field(str, description="Upstream transaction ID, idempotence Check")
#     ]
#     user_id: Annotated[str, Field(description="ID of the user to adjust")]
#     credit_type: Annotated[CreditType, Field(description="Type of credit to adjust")]
#     amount: Annotated[
#         Decimal, Field(description="Amount to adjust (positive or negative)")
#     ]
#     note: Annotated[str, Field(description="Required explanation for the adjustment")]

intentkit/models/chat.py:385

            datetime: lambda v: v.isoformat(timespec="milliseconds"),

intentkit/models/chat.py:562

  • This method requires 1 positional argument, whereas overridden ChatCreate.lambda requires 0.
        json_encoders={datetime: lambda v: v.isoformat(timespec="milliseconds")},

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


# Get the latest agent from create_or_update
latest_agent, agent_data = await deploy_agent(agent_id, agent, subject)
latest_agent, agent_data = await deploy_agent(agent_id, agent, "admin")
Copy link

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Variable latest_agent is not used.

Copilot uses AI. Check for mistakes.

# Get the latest agent from create_or_update
latest_agent, agent_data = await deploy_agent(agent_id, agent, subject)
latest_agent, agent_data = await deploy_agent(agent_id, agent, "admin")
Copy link

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Variable agent_data is not used.

Suggested change
latest_agent, agent_data = await deploy_agent(agent_id, agent, "admin")
await deploy_agent(agent_id, agent, "admin")

Copilot uses AI. Check for mistakes.
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