fix: Adding defineConfig() to cspell-types

[movahhedi]

Closes #5358

[Jason3S]

@movahhedi,

Two things:

1. The tests are failing because the API changed without updating the snapshot. The purpose of the test is to help catch accidental changes to the API.
   To fix:

   cd packages/cspell-types
   pnpm t -- -u

2. There should be a unit test for the code. Something like:
   packages/cspell-types/src/defineConfig.test.ts

   import { describe, expect, test } from 'vitest';
   
   import { defineConfig } from './defineConfig.js';
   
   describe('defineConfig', () => {
       test('defineConfig is a pure pass-through', () => {
           const config = { name: 'test' };
           Object.freeze(config);
           expect(defineConfig(config)).toBe(config);
       });
   });

[movahhedi]

Done.

[movahhedi]

The purpose of the test is to help catch accidental changes to the API.
To fix:

cd packages/cspell-types
pnpm t -- -u

I suggest adding this to package.json's scripts.

[Jason3S]

@movahhedi,

The purpose of the test is to help catch accidental changes to the API.
To fix:

cd packages/cspell-types
pnpm t -- -u

I suggest adding this to package.json's scripts.

I can see the point. It hadn't been added because the API doesn't change very often.

Looks like it is still breaking in other modules.

If you run the following from the root, it will show where the tests break:

pnpm ib
pnpm t

It is most likely in packages/cspell-lib and packages/cspell.

[Jason3S]

Done.

Thank you.

[Jason3S]

@movahhedi,

There was a script in the top level package.json file that I had forgotten about: test:update-snapshots. I tried to update your PR directly, but I wasn't able to push the changes.

Replaced by #5399

[movahhedi]

Very well.
Thanks.

In these little occasions, ping me, I'll try to be quick.

[Jason3S]

@movahhedi,

Thank you for the PR.