Container resolve does work anymore with default parameters since v4.4

Describe the bug DI does not work properly when using default values.

Error: Cannot inject the dependency at position #0 of "Target" constructor. Reason:
    TypeInfo not known for "Object"

To Reproduce Build and run main.

import "reflect-metadata";
import {container, injectable} from "tsyringe";

@injectable()
class Target {
  constructor(subject = "hello world") {}
}

function main() {
    const t = container.resolve(Target);
}

Expected behavior t correctly resolved with t.subject == “hello world”

Version: tsyringe v4.4

[RaulGF92]

Yeah, having issues with my @singleton class:

import { singleton } from 'tsyringe';

@singleton()
export class Config {
  public readonly region: string;
  constructor(
    region?: string,
  ) {
    this.region = region || process.env.REGION || 'localhost';
  }
}

Basically I need the optional value outright ignored outside of testing, unit tests call the Config file with the constructor and add it to the container.

That is your solution using Tsyring but if you need to create runtime Config instances I don’t recommend you use singleton or Tsyringe, use a simple class.

import { singleton, inject } from 'tsyringe';

container.register('ConfigRegion', {
    useValue: process.env.REGION || 'localhost'
});

@singleton()
export class Config {
  public readonly region: string;
  constructor(
    @inject("ConfigRegion") region?: string,
  ) {
    this.region = region;
  }
}

[RaulGF92]

Why is it closed but it’s not the best approach?

This is how I solved the problem indeed this solution is in the part of documentation…

https://github.com/microsoft/tsyringe#injecting-primitive-values-named-injection

import { container, inject, injectable } from 'tsyringe';
import config from 'config';
import AWS from 'aws-sdk';

container.register('SqsClientQueueUrl', {
    useValue: <string> config.get('cu-enablement-queue-url')
});

@injectable()
export default class SqsClient {
    constructor(
        private sqs: AWS.SQS = new AWS.SQS(),
        @inject('SqsClientQueueUrl') private queueUrl: string
    ) {
    }

    sendMessage(message) {
        const params = {
            MessageBody: JSON.stringify(message),
            QueueUrl: <string> this.queueUrl
        };
        return this.sqs.sendMessage(params).promise();
    }
}

But I don’t think this is the best approach because you are forced to make an extern initialization… Hopefully, the dev team will implement the logical behaviour and we won’t need to use that.