Ref(api-specification): refactoring folder structure for occupation group relations api-specification definetion by C5rogers · Pull Request #483 · tabiya-tech/taxonomy-model-application

C5rogers · 2026-03-04T08:42:48Z

This pull request have a change on the api-specification of OccupationGroup for relations api separeting based on there hierarchy

…roup relations api-specification definetion

Copilot

Pull request overview

This PR refactors the api-specifications for OccupationGroup relation endpoints by splitting relation-specific pieces into relations/parent and relations/children folders and updating imports/exports accordingly.

Changes:

Update relation schema/test imports to reflect the new relations/* folder structure.
Add new relation-specific enum modules (relations/parent/enum.ts, relations/children/enum.ts) and export them from occupationGroup/index.ts.
Update the occupationGroup module export snapshot to include the new enum exports.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
backend/globalConfig.json	Adds a hard-coded local MongoDB URI config file.
api-specifications/src/esco/occupationGroup/relations/parent/schema.GET.parent.response.ts	Fixes `_baseResponseSchema` import path after moving under `relations/parent`.
api-specifications/src/esco/occupationGroup/relations/parent/schema.GET.parent.response.test.ts	Updates imports to reference `occupationGroup` root module/enums/regex after refactor.
api-specifications/src/esco/occupationGroup/relations/parent/enum.ts	Introduces parent-relation enum namespace and error codes.
api-specifications/src/esco/occupationGroup/relations/children/schema.GET.child.response.ts	Fixes `_baseChildrenResponseSchema` import path after moving under `relations/children`.
api-specifications/src/esco/occupationGroup/relations/children/schema.GET.child.response.test.ts	Updates imports to reference `occupationGroup` root module/enums/regex after refactor.
api-specifications/src/esco/occupationGroup/relations/children/enum.ts	Introduces children-relation enum namespace and error codes.
api-specifications/src/esco/occupationGroup/index.ts	Exposes moved schemas and newly added `ParentEnums`/`ChildrenEnums` via the main module export.
api-specifications/src/esco/occupationGroup/snapshots/index.test.ts.snap	Snapshot update to include the new exported enum namespaces.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-04T08:50:44Z

backend/globalConfig.json

@@ -0,0 +1 @@
+{"mongoUri":"mongodb://127.0.0.1:24780/"}


backend/globalConfig.json introduces a hard-coded local MongoDB URI. This file doesn’t appear to be referenced anywhere in backend/src (config is read from env elsewhere), and committing environment-specific connection strings can cause confusion in CI/prod. Consider removing this file from the repo (or replacing it with a documented example like globalConfig.example.json) and loading the URI from environment variables; if it must exist locally, add it to .gitignore.

Suggested change

{"mongoUri":"mongodb://127.0.0.1:24780/"}

{"mongoUri":"YOUR_MONGODB_URI_HERE"}

Copilot · 2026-03-04T08:50:44Z

api-specifications/src/esco/occupationGroup/relations/parent/enum.ts

+
+namespace OccupationGroupParentEnums {
+  export enum ObjectTypes {
+    ISCOGroup = CommonGroupTypes.ISCOGroup,
+    LocalGroup = CommonGroupTypes.LocalGroup,
+  }
+  export namespace Relations {
+    export namespace Parent {
+      export enum ObjectTypes {
+        ISCOGroup = CommonGroupTypes.ISCOGroup,
+        LocalGroup = CommonGroupTypes.LocalGroup,
+      }
+    }
+    export namespace Children {
+      export enum ObjectTypes {
+        ISCOGroup = CommonGroupTypes.ISCOGroup,
+        LocalGroup = CommonGroupTypes.LocalGroup,
+        ESCOOccupation = CommonGroupTypes.ESCOOccupation,
+        LocalOccupation = CommonGroupTypes.LocalOccupation,
+      }
+    }
+  }


This new file duplicates constants already defined in api-specifications/src/esco/occupationGroup/enums.ts (e.g., Relations.Parent.ObjectTypes and Relations.Children.ObjectTypes) and also duplicates Relations.Children.ObjectTypes again in OccupationGroupChildrenEnums. Having multiple sources of truth for the same enum values/error codes is likely to drift over time. Consider consolidating to a single definition (e.g., export the relation enums from one module and import/re-export them where needed) and removing the duplicated enum blocks here.

Suggested change

namespace OccupationGroupParentEnums {

export enum ObjectTypes {

ISCOGroup = CommonGroupTypes.ISCOGroup,

LocalGroup = CommonGroupTypes.LocalGroup,

}

export namespace Relations {

export namespace Parent {

export enum ObjectTypes {

ISCOGroup = CommonGroupTypes.ISCOGroup,

LocalGroup = CommonGroupTypes.LocalGroup,

}

}

export namespace Children {

export enum ObjectTypes {

ISCOGroup = CommonGroupTypes.ISCOGroup,

LocalGroup = CommonGroupTypes.LocalGroup,

ESCOOccupation = CommonGroupTypes.ESCOOccupation,

LocalOccupation = CommonGroupTypes.LocalOccupation,

}

}

}

import OccupationGroupEnums from "../../enums";

namespace OccupationGroupParentEnums {

export enum ObjectTypes {

ISCOGroup = CommonGroupTypes.ISCOGroup,

LocalGroup = CommonGroupTypes.LocalGroup,

}

export import Relations = OccupationGroupEnums.Relations;

Copilot · 2026-03-04T08:50:45Z

api-specifications/src/esco/occupationGroup/relations/children/enums.ts

+import { ObjectTypes as CommonGroupTypes } from "../../../common/objectTypes";
+
+namespace OccupationGroupChildrenEnums {


The filename enum.ts is inconsistent with the existing convention in this repo for enum modules (commonly enums.ts, including other relation enum modules). Renaming to enums.ts would improve discoverability and consistency across api-specifications/src/esco/**/relations/** folders.

Copilot · 2026-03-04T08:50:45Z

api-specifications/src/esco/occupationGroup/index.ts

 namespace OccupationGroupAPISpecs {
  export import Enums = OccupationGroupEnums;
  export import Types = OccupationGroupTypes;
  export import Constants = OccupationGroupConstants;
  export import Schemas = OccupationGroupSchemas;
  export import Patterns = OccupationGroupRegexes;
+  export import ParentEnums = OccupationGroupParentEnums;
+  export import ChildrenEnums = OccupationGroupChildrenEnums;


OccupationGroupAPISpecs now exports both Enums (which already contains Relations.Parent/Children enums) and the new ParentEnums/ChildrenEnums. This creates two parallel public enum hierarchies for the same concepts, which is easy for API consumers to misuse and hard to keep in sync. Consider either (1) moving the relation enums out of ./enums.ts and re-exporting them from a single place, or (2) not exporting ParentEnums/ChildrenEnums and keeping Enums.Relations.* as the only supported path.

Copilot · 2026-03-04T08:50:45Z

backend/globalConfig.json

@@ -0,0 +1 @@
+{"mongoUri":"mongodb://127.0.0.1:24780/"}


This change adds backend/globalConfig.json, but the PR description is about refactoring the api-specifications folder structure for OccupationGroup relations. If globalConfig.json is unrelated, it should be removed from this PR to keep the change scoped and easier to review.

irumvanselme · 2026-03-04T09:07:31Z

backend/globalConfig.json

This file, should not be commited

Copilot

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-04T13:13:51Z

api-specifications/src/esco/occupationGroup/relations/parent/enums.ts

+        export enum ErrorCodes {
+          INVALID_LIMIT_PARAMETER = "INVALID_LIMIT_PARAMETER",
+          INVALID_NEXT_CURSOR_PARAMETER = "INVALID_NEXT_CURSOR_PARAMETER",
+        }


OccupationGroupParentEnums duplicates values that already exist in OccupationGroupEnums (e.g. object types and the shared 400 error codes). This creates a drift risk if one set is updated without the other. Prefer re-exporting the existing enums (or a shared/common enum) instead of redefining the same members here.

Suggested change

export enum ErrorCodes {

INVALID_LIMIT_PARAMETER = "INVALID_LIMIT_PARAMETER",

INVALID_NEXT_CURSOR_PARAMETER = "INVALID_NEXT_CURSOR_PARAMETER",

}

export import ErrorCodes = OccupationGroupEnums.GET.Response.Status400.ErrorCodes;

Copilot · 2026-03-04T13:13:51Z

api-specifications/src/esco/occupationGroup/relations/parent/enums.ts

+    ISCOGroup = CommonGroupTypes.ISCOGroup,
+    LocalGroup = CommonGroupTypes.LocalGroup,
+  }
+  export import Relations = OccupationGroupEnums.Relations;


export import Relations = OccupationGroupEnums.Relations; re-exports all relation enums (including children) from within the parent-specific enum module, which makes this module’s API broader and potentially misleading. Consider removing this re-export or narrowing it to only the parent-related types so ParentEnums stays focused on the parent relation.

Suggested change

export import Relations = OccupationGroupEnums.Relations;

Copilot · 2026-03-04T13:13:52Z

api-specifications/src/esco/occupationGroup/relations/children/enums.ts

+  export enum ObjectTypes {
+    ISCOGroup = CommonGroupTypes.ISCOGroup,
+    LocalGroup = CommonGroupTypes.LocalGroup,
+    ESCOOccupation = CommonGroupTypes.ESCOOccupation,
+    LocalOccupation = CommonGroupTypes.LocalOccupation,
+  }
+  export namespace GET {
+    export namespace Response {
+      export namespace Status400 {
+        export enum ErrorCodes {
+          INVALID_LIMIT_PARAMETER = "INVALID_LIMIT_PARAMETER",
+          INVALID_NEXT_CURSOR_PARAMETER = "INVALID_NEXT_CURSOR_PARAMETER",
+        }


OccupationGroupChildrenEnums repeats enum members that already exist in OccupationGroupEnums.Relations.Children.ObjectTypes and the shared 400 error codes. Duplicating these constants across modules increases maintenance overhead and can lead to inconsistent values over time. Prefer re-exporting from a single source of truth where possible.

Ref(api-specification): refactoring folder structure for occupation g…

08cf000

…roup relations api-specification definetion

C5rogers requested review from HabtamuTesafaye, irumvanselme and rav3n11 March 4, 2026 08:42

C5rogers self-assigned this Mar 4, 2026

C5rogers added the documentation Improvements or additions to documentation label Mar 4, 2026

C5rogers requested a review from Copilot March 4, 2026 08:46

Copilot started reviewing on behalf of C5rogers March 4, 2026 08:46 View session

Copilot AI reviewed Mar 4, 2026

View reviewed changes

irumvanselme reviewed Mar 4, 2026

View reviewed changes

Fix(api-specification|backend): folder structure review fix for copilot

1df6b0e

C5rogers requested a review from Copilot March 4, 2026 13:07

Copilot started reviewing on behalf of C5rogers March 4, 2026 13:07 View session

Copilot AI reviewed Mar 4, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ref(api-specification): refactoring folder structure for occupation group relations api-specification definetion#483

Ref(api-specification): refactoring folder structure for occupation group relations api-specification definetion#483
C5rogers wants to merge 2 commits intomainfrom
fix/refactoring_folder_structure_for_occupation_group_api_specification

C5rogers commented Mar 4, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

irumvanselme Mar 4, 2026

Uh oh!

C5rogers Mar 9, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		@@ -0,0 +1 @@
		{"mongoUri":"mongodb://127.0.0.1:24780/"} No newline at end of file

	{"mongoUri":"mongodb://127.0.0.1:24780/"}
	{"mongoUri":"YOUR_MONGODB_URI_HERE"}

		import { ObjectTypes as CommonGroupTypes } from "../../../common/objectTypes";

		namespace OccupationGroupChildrenEnums {

Conversation

C5rogers commented Mar 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

irumvanselme Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

C5rogers Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants