Skip to content

fix(calm-hub): make mongo initialization log consistent#2286

Merged
jpgough-ms merged 3 commits intofinos:mainfrom
markscott-ms:fix/init-mongo
Mar 31, 2026
Merged

fix(calm-hub): make mongo initialization log consistent#2286
jpgough-ms merged 3 commits intofinos:mainfrom
markscott-ms:fix/init-mongo

Conversation

@markscott-ms
Copy link
Copy Markdown
Contributor

@markscott-ms markscott-ms commented Mar 30, 2026

Description

Recent additions of controls, domains and decorators led to some inconsistent logging during MongoDB initialisation, and a JSON syntax in /calm/architecture/calm-2.json

This PR fixes that.

No functionality change - just print statements.

$ mongosh mongo/init-mongo.js 
  ✅ Using database: calmSchemas
=== Counters ===
  - patternStoreCounter already exists, no initialization needed
  - architectureStoreCounter already exists, no initialization needed
  - adrStoreCounter already exists, no initialization needed
  - standardStoreCounter already exists, no initialization needed
  - flowStoreCounter already exists, no initialization needed
  - userAccessStoreCounter already exists, no initialization needed
  - controlStoreCounter already exists, no initialization needed
  - decoratorStoreCounter already exists, no initialization needed
=== Schemas ===
  ❌ Schema directory not found at /calm/release, skipping
  ❌ Set CALM_SCHEMA_BASE_PATH environment variable to load schemas from a different location
...

Type of Change

  • 🐛 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)
  • 📚 Documentation update
  • 🎨 Code style/formatting changes
  • ♻️ Refactoring (no functional changes)
  • ⚡ Performance improvements
  • ✅ Test additions or updates
  • 🔧 Chore (maintenance, dependencies, CI, etc.)

Affected Components

  • CLI (cli/)
  • Schema (calm/)
  • CALM AI (calm-ai/)
  • CALM Hub (calm-hub/)
  • CALM Hub UI (calm-hub-ui/)
  • CALM Server (calm-server/)
  • CALM Widgets (calm-widgets/)
  • Documentation (docs/)
  • Shared (shared/)
  • VS Code Extension (calm-plugins/vscode/)
  • Dependencies
  • CI/CD

Commit Message Format ✅

Testing

  • I have tested my changes locally
  • I have added/updated unit tests
  • All existing tests pass

Checklist

  • My commits follow the conventional commit format
  • I have updated documentation if necessary
  • I have added tests for my changes (if applicable)
  • My changes follow the project's coding standards

aamanrebello
aamanrebello previously approved these changes Mar 30, 2026
Comment thread calm-hub/mongo/init-mongo.js Outdated
aamanrebello
aamanrebello previously approved these changes Mar 30, 2026
Copy link
Copy Markdown
Contributor

@aamanrebello aamanrebello left a comment

Choose a reason for hiding this comment

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

Looks good!

@markscott-ms
Copy link
Copy Markdown
Contributor Author

Thanks @aamanrebello . I'm going to mark this PR as draft as I'll need to make an update after @harveymmaunders 's PR #2285 is merged.

@markscott-ms markscott-ms marked this pull request as draft March 30, 2026 18:16
@github-actions github-actions bot added the calm Affects calm folder (possibly the schema) label Mar 30, 2026
@markscott-ms markscott-ms marked this pull request as ready for review March 30, 2026 20:11
@markscott-ms markscott-ms requested review from a team as code owners March 30, 2026 20:11
@jpgough-ms jpgough-ms merged commit 78a48ab into finos:main Mar 31, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

calm Affects calm folder (possibly the schema) calm-hub Affects `calm-hub` config

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants