Skip to content

fix(tests): revert OASF schema version from 1.1.0 back to 1.0.0 - #1815

Merged
paralta merged 2 commits into
mainfrom
refactor/revert-oasf-110
Jul 15, 2026
Merged

fix(tests): revert OASF schema version from 1.1.0 back to 1.0.0#1815
paralta merged 2 commits into
mainfrom
refactor/revert-oasf-110

Conversation

@paralta

@paralta paralta commented Jul 15, 2026

Copy link
Copy Markdown
Member

The OASF 1.1.0 schema deployment at schema.oasf.outshift.com was reverted, causing the /api/1.1.0/validate/... endpoint to return 404 and breaking any push of a record with schema_version: "1.1.0". This PR undoes the schema bump introduced in #1785 and #1806, restoring all OASF references — skill taxonomy names, domain IDs, test fixtures, the OASF server image tag, and the self-describing DIR skill record — to their 1.0.0 equivalents.

@github-actions github-actions Bot added the size/XS Denotes a PR that changes 0-49 lines label Jul 15, 2026
@paralta
paralta force-pushed the refactor/revert-oasf-110 branch from ae0c476 to 7364dd8 Compare July 15, 2026 13:09
@github-actions github-actions Bot added size/S Denotes a PR that changes 50-199 lines and removed size/XS Denotes a PR that changes 0-49 lines labels Jul 15, 2026
@paralta
paralta force-pushed the refactor/revert-oasf-110 branch from 7364dd8 to 6a27270 Compare July 15, 2026 13:13
@paralta paralta changed the title refactor(dir): revert oasf schema bump to 1.1.0 fix(tests): revert OASF schema version from 1.1.0 back to 1.0.0 Jul 15, 2026
@paralta
paralta force-pushed the refactor/revert-oasf-110 branch from 6a27270 to 9340c45 Compare July 15, 2026 13:30
Comment thread tests/test_utils/oasf_110.go Outdated
{Id: 10102, Name: "natural_language_processing/natural_language_understanding/semantic_understanding"}, //nolint:mnd
{Id: 10103, Name: "natural_language_processing/natural_language_understanding/entity_recognition"}, //nolint:mnd
{Id: 20103, Name: "computer_vision/image_analysis/image_segmentation"}, //nolint:mnd
{Id: 1504, Name: "reasoning_planning/strategic_planning"}, //nolint:mnd

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.

Suggested change
{Id: 1504, Name: "reasoning_planning/strategic_planning"}, //nolint:mnd
{Id: 1504, Name: "advanced_reasoning_planning/hypothesis_generation"}, //nolint:mnd

Comment thread tests/test_utils/oasf_110.go Outdated
{Id: 10101, Name: "natural_language_processing/natural_language_understanding/contextual_comprehension"}, //nolint:mnd
{Id: 10102, Name: "natural_language_processing/natural_language_understanding/semantic_understanding"}, //nolint:mnd
{Id: 10103, Name: "natural_language_processing/natural_language_understanding/entity_recognition"}, //nolint:mnd
{Id: 20103, Name: "computer_vision/image_analysis/image_segmentation"}, //nolint:mnd

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.

does not exist

Suggested change
{Id: 20103, Name: "computer_vision/image_analysis/image_segmentation"}, //nolint:mnd
{Id: 201, Name: "images_computer_vision/image_segmentation"}, //nolint:mnd

Signed-off-by: Catarina Paralta <clouropa@cisco.com>
@paralta
paralta force-pushed the refactor/revert-oasf-110 branch from 9340c45 to ac660e3 Compare July 15, 2026 13:45
Signed-off-by: Catarina Paralta <clouropa@cisco.com>
@paralta
paralta marked this pull request as ready for review July 15, 2026 15:24
@paralta
paralta requested a review from a team as a code owner July 15, 2026 15:24
@paralta
paralta merged commit 7d3135b into main Jul 15, 2026
31 checks passed
@paralta
paralta deleted the refactor/revert-oasf-110 branch July 15, 2026 15:27
@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 50-199 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants