Skip to content

Conversation

@vaibhavbhalla2505
Copy link

No description provided.

@vaibhavbhalla2505 vaibhavbhalla2505 self-assigned this Jul 16, 2025
@vaibhavbhalla2505 vaibhavbhalla2505 changed the title add CLI page docs(arc-api): add CLI page Jul 16, 2025
@@ -0,0 +1,3 @@
title: Packages
nav:

Choose a reason for hiding this comment

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

??

Copy link
Author

Choose a reason for hiding this comment

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

@Tyagi-Sunny Sir,this adds a single navigation item labeled "CLI" that links to the file cli.md .

@yeshamavani
Copy link
Contributor

@vaibhavbhalla2505 can you show me how this looks

Copy link

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 adds CLI documentation to the arc-api-docs by introducing a new packages section with CLI command reference material. The documentation provides comprehensive information about the @sourceloop/cli package, including installation instructions and detailed command descriptions.

  • Adds CLI documentation page with installation and command reference
  • Creates navigation structure for packages section
  • Documents 8 CLI commands: autocomplete, cdk, extension, help, mcp, microservice, scaffold, and update

Reviewed changes

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

File Description
docs/arc-api-docs/packages/pages.yml Adds navigation configuration for the packages section with CLI as the first entry
docs/arc-api-docs/packages/cli.md Comprehensive CLI documentation with installation instructions, command usage, options, and examples

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

OPTIONS
-a, --applicationClassName
-d, --dir
-i, --iac=(lambda)
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

Line 58 has an unnecessary trailing space after "(lambda)" that should be removed for consistency. The other option lines in the documentation (lines 56, 57, 59-61) don't have trailing spaces, so this line should match that pattern.

Suggested change
-i, --iac=(lambda)
-i, --iac=(lambda)

Copilot uses AI. Check for mistakes.
-a, --applicationClassName
-d, --dir
-i, --iac=(lambda)
-o, --overwriteDockerfile
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

Line 59 has an unnecessary trailing space after "--overwriteDockerfile" that should be removed for consistency. The other option lines in the documentation don't have trailing spaces, so this line should match that pattern.

Suggested change
-o, --overwriteDockerfile
-o, --overwriteDockerfile

Copilot uses AI. Check for mistakes.
--cwd
--description
--helmPath
--help
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

Line 136 has an unnecessary trailing space after "--help" that should be removed for consistency. The other option lines in the documentation don't have trailing spaces, so this line should match that pattern.

Suggested change
--help
--help

Copilot uses AI. Check for mistakes.
--description
--helmPath
--help
--integrateWithBackstage
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

Line 137 has an unnecessary trailing space after "--integrateWithBackstage" that should be removed for consistency. The other option lines in the documentation don't have trailing spaces, so this line should match that pattern.

Suggested change
--integrateWithBackstage
--integrateWithBackstage

Copilot uses AI. Check for mistakes.
--help
--integrateWithBackstage
--issuePrefix
--jenkinsfile
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

Line 139 has an unnecessary trailing space after "--jenkinsfile" that should be removed for consistency. The other option lines in the documentation don't have trailing spaces, so this line should match that pattern.

Suggested change
--jenkinsfile
--jenkinsfile

Copilot uses AI. Check for mistakes.
@@ -0,0 +1,152 @@
@sourceloop/cli /

# @sourceloop-cli
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

The heading # @sourceloop-cli is inconsistent with the package name shown elsewhere in the document. The package name is @sourceloop/cli (with a forward slash), as seen in line 1, line 12, and line 96. The heading should be updated to match the actual package name @sourceloop/cli for consistency.

Suggested change
# @sourceloop-cli
# @sourceloop/cli

Copilot uses AI. Check for mistakes.

## Installation

```shell
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

The word "shell" is incorrectly capitalized as "Shell" in the code block header. This should be lowercase "shell" to be consistent with standard Markdown code block syntax.

Copilot uses AI. Check for mistakes.
- sl autocomplete
- sl cdk
- sl extension
- sl help
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

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

The trailing space on line 23 should be removed. This appears after "sl help" and before the line break, which is inconsistent with the formatting of other command list items.

Suggested change
- sl help
- sl help

Copilot uses AI. Check for mistakes.
@vaibhavbhalla2505
Copy link
Author

@vaibhavbhalla2505 can you show me how this looks

Screen.Recording.2025-12-12.at.11.59.51.AM.mov

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.

5 participants