Amphtml: [Master feature] TypeScript in AMP

[choumx]

Add TypeScript support to AMP and allow new extensions to be written in TypeScript.

Motivation

Given the increasing maturity and popularity of TypeScript and potential developer productivity benefits, we'd like to experiment with TypeScript support in AMP.

Requirements

• Supports existing toolchain, particularly Closure Compiler usage
• Supports piecemeal adoption (with a rollback plan in case this I2I is rejected)

Proposal

1. Experiment: Allow new extensions to be written in TypeScript, which will be transpiled to Closure-annotated JS via tsickle as a preprocess to our normal compilation pipeline.
2. Gather data: Ship at least one new AMP extension written in TypeScript and collate pros/cons of development experience. Survey AMP community on TypeScript vs. JavaScript preferences.
3. Convert: If gathered data is supportive, add support for incremental transpilation for remaining source and convert existing JS to TS file-by-file.

[choumx]

/cc @kristoferbaxter @cramforce @rsimha @erwinmombay @jridgewell

[cramforce]

I wish I had done this July 22nd 2015 :)

[choumx]

Everyone, please fill out the public preliminary interest survey! 🙏

[choumx]

Overall:
~15% disapprove, ~20% undecided and over 60% approve.

Positive sentiment:
Static typing should yield better productivity and less buggy code.

Negative sentiment:
Concerns about TS's longevity, learning curve, and opportunity cost of code migration.

See full survey results.

[ampprojectbot]

This issue seems to be in Pending Triage for awhile. @danielrozenberg Please triage this to an appropriate milestone.

[achrinza]

Just checking in; is there any update on this? Currently this is causing friction in adopting Next.js with Typescript.

[dkolba]

@achrinza use this meanwhile

[choumx]

This experiment hasn't been prioritized yet. Deferring to @ampproject/wg-infra for scheduling.

[rcebulko]

I'd be interested in taking a stab at Closure/Typescript interactions, what the migration process could look like, etc. Infra team has discussed using build-system as a jumping off point for the migration, since it doesn't reach the release and so would have less risk of causing a regression during migration, but would allow us to set up the main infra around building, running, and testing TS+JS

[fx69005]

someone have news for this feature ?

[choumx]

There's insufficient staffing for this for the foreseeable future. Closing.

[SiaQnbr]

Isn't there any news about the amp on ts still?

[samouri]

@SiaQnbr: no news beyond what you see here

[markhughes]

@choumx that does not seem like a decent reason for closure of a ticket, especially for something that could be automated on publish w/ something like https://github.com/urish/typewiz to at least provide _something_ to devs.

[rcebulko]

@markhughes It seems like you have some familiarity with the relevant tooling here, so if you have an idea for a useful and non-intrusive way to address this, feel free to put a PR together and add myself/@ampproject/wg-infra to take a look at it. There's some added complication with migrating to TypeScript because of the various compilation and optimization steps that take place, which makes a transition of runtime code a bit risky, though we've had discussions about migrating some build-system code at some point. The work involved poses a not insignificant opportunity cost for the team to work on it, which I believe is part of why this feature request was closed.

[markhughes]

My comment was not about migrating to typescript on the spot. That was about adding typescript types.

For example: https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/auth0

This would allow developers using TypeScript to use the package without the package actually migrating to typescript.

[samouri]

@markhughes: Can you explain more? In general, developers using AMP to build websites don't directly consume AMP JS APIs. They use our Web Components instead, e.g.

<amp-list> ... </amp-list>

I don't think TypeScript does html type-checking

[yassinebridi]

@samouri But there is frameworks that support amp, like Nextjs:
https://nextjs.org/docs/advanced-features/amp-support/introduction
or Nuxtjs:
https://github.com/nuxt-community/amp-module