Test for deeply nested through joins.

[SippieCup]

Note: This also add a sequelize 7 compatibility wrapper for sscce.

Unit test for minification support on joins and new join query generation.

[Copilot]

Pull Request Overview

This PR adds comprehensive unit tests for deeply nested join operations in Sequelize 7, focusing on minification support and new join query generation. It also introduces Sequelize 7 compatibility wrappers.

• Replaces simple SSCCE example with complex nested join test cases involving multiple models
• Adds Sequelize 7 configuration wrapper to properly handle dialect differences
• Tests deeply nested associations with various join requirements

Reviewed Changes

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

File	Description
src/sscce-sequelize-7.ts	Complete rewrite with complex model definitions and deeply nested join test cases
dev/wrap-options.ts	Adds Sequelize 7 configuration wrapper with proper option transformation
dev/create-sequelize-instance.ts	Updates to use new Sequelize 7 wrapper functionality

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Missing association decorator for the location property. Consider adding the corresponding BelongsTo decorator to maintain consistency with other model relationships.

Suggested change

	declare locationId: number;
	declare locationId: number;
	@BelongsTo(() => Location, {
	foreignKey: "locationId",
	inverse: "systems",
	})

[Copilot]

Missing association decorator for the system property. Consider adding the corresponding BelongsTo decorator to maintain consistency with other model relationships.

Suggested change

	@BelongsTo(() => System, {
	foreignKey: "systemId",
	inverse: "fuelDeliveries",
	})

[Copilot]

Missing association decorators for the customer and location properties. Consider adding the corresponding BelongsTo decorators to maintain consistency with other model relationships.

[Copilot]

Debug console.log statement should be removed from production test code. This appears to be leftover debugging code.

Suggested change

console.log(system);

[Copilot]

Using 'any' as the generic type parameter reduces type safety. Consider using a more specific type or making it generic to preserve type information.

Suggested change

	export function wrapOptionsV7(options: Partial<Sequelize7Options<any>> = {}) {
	export function wrapOptionsV7<M extends object = any>(options: Partial<Sequelize7Options<M>> = {}) {

[Copilot]

Using 'as any' type assertion bypasses type safety. Consider properly typing the config transformation or using a type-safe approach to handle the configuration mapping.

Suggested change

	const config = CiDbConfigs[dialect as keyof typeof CiDbConfigs] as any;
	const config = CiDbConfigs[dialect as keyof typeof CiDbConfigs] as Partial<Sequelize7Options<any>>;