Skip to content

feat:cext-4860 added standalone commands - #13

Merged
simrangulati merged 13 commits into
epic/abdb-implementationfrom
feat/cext-4860
Jul 11, 2025
Merged

feat:cext-4860 added standalone commands#13
simrangulati merged 13 commits into
epic/abdb-implementationfrom
feat/cext-4860

Conversation

@simrangulati

@simrangulati simrangulati commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

Implements the standalone commands for app builder db.

Description

To implement below commands for App Builder DB

  1. db stats
  2. db getCollectionNames
  3. db show collections
  4. db getCollectionInfos

Related Issue

Motivation and Context

It helps developers/users to get detailed information about the DB.

How Has This Been Tested?

  1. By running unit tests
  2. By linking the local version and testing commands on terminal

Screenshots (if appropriate):

image image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@simrangulati
simrangulati marked this pull request as ready for review July 8, 2025 09:27

@nofuss nofuss left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

except for a missing --json handling, looks good to me

Comment thread src/commands/app/db/getCollectionInfos.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js Outdated
Comment thread src/commands/app/db/stats.js Outdated
Comment thread src/commands/app/db/getCollectionNames.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js
Comment thread src/commands/app/db/getCollectionNames.js
Comment thread src/commands/app/db/stats.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js
Comment thread src/commands/app/db/stats.js Outdated
Comment thread src/commands/app/db/stats.js
Comment thread src/commands/app/db/stats.js Outdated
Comment thread src/commands/app/db/getCollectionInfos.js
Comment thread src/commands/app/db/getCollectionInfos.js Outdated
Comment thread src/commands/app/db/getCollectionNames.js Outdated
Comment thread src/commands/app/db/stats.js Outdated
Comment thread src/commands/app/db/stats.js
Comment thread test/commands/app/db/getCollectionInfos.test.js
Comment thread test/commands/app/db/getCollectionInfos.test.js
Comment thread test/commands/app/db/getCollectionInfos.test.js
Comment thread test/commands/app/db/getCollectionInfos.test.js
Comment thread test/commands/app/db/getCollectionNames.test.js Outdated
Comment thread test/commands/app/db/getCollectionNames.test.js Outdated
Comment thread test/commands/app/db/stats.test.js
Comment thread test/commands/app/db/stats.test.js
Comment thread test/commands/app/db/stats.test.js

@nofuss nofuss left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, thought I don't understand the 'region' flag check.

Comment thread test/commands/app/db/getCollectionNames.test.js
@simrangulati
simrangulati merged commit 4b961e1 into epic/abdb-implementation Jul 11, 2025
2 checks passed
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