Skip to content

feat: utilize moon for monorepo management - #422

Open
gebhardtr wants to merge 5 commits into
oracle:mainfrom
gebhardtr:rigebha/moon
Open

feat: utilize moon for monorepo management#422
gebhardtr wants to merge 5 commits into
oracle:mainfrom
gebhardtr:rigebha/moon

Conversation

@gebhardtr

@gebhardtr gebhardtr commented Aug 21, 2026

Copy link
Copy Markdown
Member

Description

Make management of poloyglot servers easier with the moon monorepo framework.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • 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 not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • Firmware version:
  • Hardware:
  • Toolchain:
  • SDK:

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 21, 2026
@@ -0,0 +1,2 @@
dependsOn:
- 'common'

@gebhardtr gebhardtr Aug 21, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

required until moon can handle version ranges, like the following, properly in the project dependencies:

oracle-mcp-common>=0.1.2,<0.2.0

@gebhardtr
gebhardtr marked this pull request as ready for review August 21, 2026 16:41

@AlaaShaker AlaaShaker left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's a small change but might be worth fixing before we merge.

Comment thread moon.yml Outdated
@gebhardtr
gebhardtr force-pushed the rigebha/moon branch 2 times, most recently from 303458e to 23199a2 Compare August 26, 2026 16:06
@gebhardtr gebhardtr closed this Aug 26, 2026
@gebhardtr gebhardtr reopened this Aug 26, 2026
@gebhardtr gebhardtr closed this Aug 26, 2026
@gebhardtr gebhardtr reopened this Aug 26, 2026
Comment thread scripts/generate_package_metadata.py Outdated

@dustin-sale dustin-sale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Almost there, the build step in the Makefile will end up passing the common package to the same generate_package_metadata.py script as the rest of the mcp servers. This script does not support the common package properly.

@gebhardtr
gebhardtr marked this pull request as draft August 27, 2026 16:00
@gebhardtr

gebhardtr commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

#427 simplifying package version handling in order to simplify this PR

@dustin-sale dustin-sale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sigh, one more thing. Can you add the moon.yml files to the exclude section under [tool.hatch.build.targets.sdist] in each pyproject.toml file?

@gebhardtr gebhardtr mentioned this pull request Sep 1, 2026
14 tasks
Signed-off-by: Richard Gebhardt <richard.gebhardt@oracle.com>
Signed-off-by: Richard Gebhardt <richard.gebhardt@oracle.com>
Signed-off-by: Richard Gebhardt <richard.gebhardt@oracle.com>
Signed-off-by: Richard Gebhardt <richard.gebhardt@oracle.com>
@gebhardtr
gebhardtr marked this pull request as ready for review September 3, 2026 16:50
Signed-off-by: Richard Gebhardt <richard.gebhardt@oracle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants