Skip to content

chore: Update to ORD 1.16#481

Draft
mlakov wants to merge 2 commits into
mainfrom
chore/update-to-ord-1-16
Draft

chore: Update to ORD 1.16#481
mlakov wants to merge 2 commits into
mainfrom
chore/update-to-ord-1-16

Conversation

@mlakov
Copy link
Copy Markdown
Contributor

@mlakov mlakov commented Jun 3, 2026

Update ORD Version to 1.16

Chore

🔧 Bumped the Open Resource Discovery (ORD) specification version from 1.14 to 1.16 across the codebase.

Changes

  • lib/defaults.js: Updated the openResourceDiscovery default value from "1.14" to "1.16".
  • lib/ord.js: Added baseUrl from appConfig.env.baseUrl to the generated ORD document structure.
  • package.json: Updated the @open-resource-discovery/specification dev dependency from 1.14.5 to 1.16.0.
  • __tests__/unit/__snapshots__/defaults.test.js.snap: Updated snapshot to reflect new ORD version "1.16".
  • __tests__/unit/__snapshots__/mocked-csn.test.js.snap: Updated openResourceDiscovery snapshots to "1.16".
  • __tests__/unit/__snapshots__/ord.e2e.test.js.snap: Updated all relevant openResourceDiscovery snapshot entries to "1.16".
  • __tests__/integration/__snapshots__/basic-auth.test.js.snap: Updated integration test snapshots to "1.16".
  • __tests__/integration/__snapshots__/cds-build.test.js.snap: Updated integration test snapshots to "1.16".
  • __tests__/integration/__snapshots__/mtls-auth-dev.test.js.snap: Updated integration test snapshots to "1.16".
  • __tests__/integration/__snapshots__/mtls-auth-prod.test.js.snap: Updated integration test snapshots to "1.16".
  • 🔄 Regenerate and Update Summary
PR Bot Information

Version: 1.21.1

  • Summary Prompt: Default Prompt
  • Output Template: Default Template
  • File Content Strategy: Full file content
  • Correlation ID: f950f5a6-33ca-4031-af23-c969f3f6a3e2
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened

Copy link
Copy Markdown
Contributor

@hyperspace-insights hyperspace-insights Bot left a comment

Choose a reason for hiding this comment

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

The PR is a straightforward version bump of the ORD specification from 1.14 to 1.16, with consistent snapshot updates across all test suites, plus the addition of baseUrl to the generated ORD document. The version bump itself is clean and complete. However, the baseUrl field introduced in lib/ord.js is missing a corresponding entry in MERGE_STRATEGIES, which will cause a runtime crash if any customOrdContentFile also contains a baseUrl key — this should be addressed before merging.

PR Bot Information

Version: 1.21.1

  • Agent Instructions:
  • File Content Strategy: Full file content
  • Correlation ID: f950f5a6-33ca-4031-af23-c969f3f6a3e2
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened

Comment thread lib/ord.js
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.

1 participant