Merge to main

[simonbs]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors the handling of default specifications in the projects domain. It introduces a new getDefaultSpecification function, updates multiple modules to use it, and adds the isDefault flag and defaultSpecificationName to support default selection behavior.

• Introduces getDefaultSpecification in Version.ts and updates its usage in getProjectSelectionFromPath.ts, ProjectNavigator.ts, and useProjectSelection.ts.
• Updates OpenApiSpecification and IProjectConfig to include isDefault and defaultSpecificationName respectively.
• Updates the GitHubProjectDataSource and its tests to support default specification assignment and ordering.

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/features/projects/domain/index.ts	Exposes getDefaultSpecification from Version.ts
src/features/projects/domain/getProjectSelectionFromPath.ts	Uses getDefaultSpecification to select a specification
src/features/projects/domain/Version.ts	Adds getDefaultSpecification function
src/features/projects/domain/ProjectNavigator.ts	Uses getDefaultSpecification for navigation
src/features/projects/domain/OpenApiSpecification.ts	Adds isDefault field to the specification schema
src/features/projects/domain/IProjectConfig.ts	Adds defaultSpecificationName to the project config schema
src/features/projects/data/useProjectSelection.ts	Uses getDefaultSpecification instead of direct indexing
src/features/projects/data/GitHubProjectDataSource.ts	Updates version handling to set default specifications and sorts specifications
test/projects/GitHubProjectDataSource.test.ts	Tests updated to verify correct default specification assignment and sort order
.github/dependabot.yml	Changes dependabot update interval from weekly to monthly

Files not reviewed (1)

• package.json: Language not supported

Comments suppressed due to low confidence (1)

test/projects/GitHubProjectDataSource.test.ts:210

• There appears to be a duplicate entry for 'foo-service.yml' in this test case. Confirm if duplicate specifications are expected or if this is an oversight.

{ id: "foo-service.yml",

[Copilot]

The function 'sortSpecificationsByName' is defined but never used. Consider removing it or integrating it into the version processing pipeline if sorting by specification name is intended.

Suggested change

	return {
	...version,
	specifications: version.specifications.map(spec => ({
	...spec,
	isDefault: spec.name === defaultSpecificationName
	}))
	}
	const updatedVersion = {
	...version,
	specifications: version.specifications.map(spec => ({
	...spec,
	isDefault: spec.name === defaultSpecificationName
	}))
	};
	return this.sortSpecificationsByName(updatedVersion);