Skip to content

Conversation

@MQ37
Copy link
Contributor

@MQ37 MQ37 commented May 20, 2025

closes #124

Had to fix eslint import extension error for yargs package with ignore next line since I did not find any other solution - using .js, /index.js did not work because of the @types package.

AI GENERATED:

Replace minimist with yargs for improved CLI experience

This PR replaces the minimist package with yargs to provide a more robust command-line
interface for the Apify MCP Server.

Changes include:

• Added proper help text and documentation for CLI options
• Improved argument parsing with better type safety
• Added GitHub repository link in help output
• Maintained backward compatibility with existing CLI parameters
• Legacy parameters are hidden

This change makes the server more user-friendly while maintaining the same functionality.

@MQ37 MQ37 requested a review from Copilot May 20, 2025 08:24
@github-actions github-actions bot added the t-ai Issues owned by the AI team. label May 20, 2025
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.

Pull Request Overview

This PR improves the CLI experience for the Apify MCP Server by replacing minimist with yargs for enhanced argument parsing and help documentation.

  • Replaces minimist with yargs in src/stdio.ts
  • Enhances CLI help and documentation with detailed options and a GitHub repository link
  • Updates package.json dependencies to reflect the removal of minimist and addition of yargs

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/stdio.ts Replaces minimist with yargs, adds help text and an interface for CLI arguments
package.json Updates dependency: removes minimist and related types and adds yargs and its types

MQ37 and others added 3 commits May 20, 2025 10:24
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@MQ37 MQ37 requested a review from jirispilka May 20, 2025 08:39
Copy link
Collaborator

@jirispilka jirispilka left a comment

Choose a reason for hiding this comment

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

Cool, thanks!

@MQ37 MQ37 merged commit 2fe5211 into master May 24, 2025
2 checks passed
@MQ37 MQ37 deleted the fix/cli-help branch May 24, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-ai Issues owned by the AI team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add --help cli arg

2 participants