feat: Add data type wrappers

[abmusse]

Resolves#75
Resolves#77

[kadler]

Looks OK to me. I'd probably group them by type instead of alphabetically. Might want to add aliases to the Signed* functions without the "Signed" (ie. Int, BigInt, etc).

Does the spread operator work when the value is a string instead of an object? If not, I think the functions should all return objects.

[kadler]

Not that it's case sensitive, but for consistency I think we should use lowercase f in the types returned here.

[kadler]

Seems more clear as PackedDecimal. Same goes for Zoned later.

[abmusse]

Looks OK to me. I'd probably group them by type instead of alphabetically. Might want to add aliases to the Signed* functions without the "Signed" (ie. Int, BigInt, etc).

Does the spread operator work when the value is a string instead of an object? If not, I think the functions should all return objects.

Spread operator does work on strings but not in the way we want. It essentially will split each character in the string with a space:

functionSignedBigInt(){return'20i0';}console.log('Signed Big Int:', ...SignedBigInt())// Signed Big Int: 2 0 i 0

I agree we should return an object with type key set to the return value.

[abmusse]

I think we also should we add checks to enforce length parameter was passed for

• Char
• Varchar
• LongVarchar

As is if no length is provided to these functions we end up something like this:

> const { Varchar } = require('itoolkit');
> console.log({...itoolkit.Varchar()})
{ type: 'undefineda', varying: '2' }

Check could be something like ...

functionVarchar(length){if(!length){throwTypeError('Must provide valid length for Varchar');}return{type: `${length}a`,varying: '2'};}

> const { Varchar } = require('itoolkit');
> console.log({...itoolkit.Varchar()})
Uncaught TypeError: Must provide length for Varchar

Alternatively we could have a default length or no-op when no length is passed.

[github-actions]

👋 Hi! This pull request has been marked stale due to inactivity. If no further activity occurs, it will automatically be closed.

[kadler]

Suggested change

	*3i0int8/byteDmyint83i0
	*xmlserviceC/SQLFixed-formatRPG
	*3i0int8/byteDmyint83i0

Maybe add a header?

I think it would also be nicer to space out the columns a bit and maybe put in some separator bars. The RPG example is a bit close to the "C" example.

[kadler]

Suggested change

	*32a{varying2}varcharDmychar32avarying
	*32a{varying4}varchar4Dmychar32avarying(4)
	*32a(varying=2)varcharDmychar32avarying
	*32a(varying=4)varchar4Dmychar32avarying(4)

[kadler]

Suggested change

	*32acharDmychar32a
	*32achar[32]Dmychar32a

[kadler]

Should these @returns be adjusted? They don't actually return a string.

[abmusse]

Yup these need to updated to return objects instead.