Invalid d.ts file generated from valid input

[rogierschouten]

tsc generates a .d.ts file that it subsequently cannot parse anymore.

Input file: myfile.ts

/// <reference path="./used.d.ts" />
import myUsedId = require("used");
export class foo extends myUsedId.SomeClass {
}

Input file: used.d.ts

declare module "used" {
        // note: the following does work: export import bar = require("some_used_module");  
    import bar = require("some_used_module");  
    export import SomeClass = bar.SomeClass;
}
declare module "some_used_module" {
    export class SomeClass {

   }
}

Typescript command, tsc version 1.0.3.0:

tsc -m commonjs -d -t es5 ./myfile.ts

Output file myfile.d.ts

/// <reference path="used.d.ts" />
import myUsedId = require("used");
export declare class foo extends "some_used_module".SomeClass {
}

This output file cannot be used, it generates a compile error: TS1008: Unexpected token; 'type name' expected. It does so on the quoted "some_used_module".SomeClass.

Expected output file myfile.d.ts:

/// <reference path="used.d.ts" />
import myUsedId = require("used");
export declare class foo extends myUsedId .SomeClass {
}

A workaround is exporting bar from used.d.ts. However this breaks encapsulation unnecessarily, as ALL of bar is now exposed.

[basarat]

.d.ts files aren't supported for external modules at the moment. Old codeplex issue was : https://typescript.codeplex.com/workitem/2080 AFAIK it hasn't been moved to github,

[basarat]

.d.ts files aren't supported for external modules at the moment.

It might make more sense when combined with --out : #17

[rogierschouten]

However, currently an NPM package "dts-bundle" exists for converting the tsc-generated typings to external-module-applicable typings. And that works perfectly but for this issue. If this were solved, then the use case for external modules has a proper workaround. Sounds pretty cheap to me.

[Bartvds]

dts-bundle is a huge hack that relies on crude text transforms of regular .d.ts files. This functionality should be a feature of the compiler. Having this properly solved and being able to bundle .d.ts files in npm packages would help TS adoption in node.js space a lot.

[rogierschouten]

@Bartvds I totally agree with you. And for the time being it would already be nice if the workaround worked.

[RyanCavanaugh]

Confirmed in new compiler

[sheetalkamat]

Fixed with 875d0c0

[rogierschouten]

Great, thank you! I'm looking forward to the next release.