Skip to content

Conversation

@jarugupj
Copy link
Contributor

@jarugupj jarugupj commented Oct 31, 2025

  • Improved the list output for apps
  • refactored the limit to be 20 by default for deployments and apps returned

@jarugupj jarugupj requested a review from masnwilliams October 31, 2025 17:40
@mesa-dot-dev
Copy link

mesa-dot-dev bot commented Oct 31, 2025

Mesa Description

  • Improved the list output for apps by adding a --limit flag to the app list command.
  • Standardized the default number of items returned to 20 for the app list, app history, and deploy history commands for a more consistent experience.

Description generated by Mesa. Update settings

Copy link

@mesa-dot-dev mesa-dot-dev bot left a comment

Choose a reason for hiding this comment

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

Performed full review of f66cb26...167f890

Analysis

  1. Client-side pagination in cmd/app.go fetches all data before filtering, which won't scale well with large datasets
  2. While standardizing pagination limits to 20 entries improves typical performance, the approach is inconsistent by mixing client-side and server-side pagination strategies

Tip

Help

Configure your agents

Mesa Docs

Slash Commands:

  • /review - Request a full code review
  • /review latest - Review only changes since the last review
  • /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
  • /help - Get help with Mesa commands and configuration options

2 files reviewed | 0 comments | Edit Agent Settings

Copy link
Contributor

@masnwilliams masnwilliams left a comment

Choose a reason for hiding this comment

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

lgtm

@jarugupj jarugupj merged commit 29988ae into main Oct 31, 2025
2 checks passed
@jarugupj jarugupj deleted the phani/pagination-update-deploy-and-apps branch October 31, 2025 21:08
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.

3 participants