Skip to content

Add TypeScript Go (ts-go) testing support with @typescript/native-preview #2145

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
Copilot wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Add TypeScript Go (ts-go) testing support with @typescript/native-preview #2145

Copilot wants to merge 6 commits into from
+323 −0

Conversation

Copy link

Copilot AI commented Oct 28, 2025 •
edited
Loading

✅ Test de TypeScript Go (ts-go) complété avec succès !

  • Identifier le package officiel @typescript/native-preview de Microsoft
  • Ajouter @typescript/native-preview (v7.0.0-dev.20251027.1) comme dépendance
  • Garder TypeScript standard 5.7 pour la compilation JavaScript normale
  • Créer un workflow CI dédié pour tester TypeScript Go (tsgo-test.yml)
  • Ajouter un script npm type-check:tsgo pour tester avec TypeScript Go
  • Mettre à jour le workflow CI principal pour afficher la version TypeScript
  • Tester la compilation avec tsgo (Version 7.0.0-dev.20251027.1 ✅)
  • Vérifier que le build standard fonctionne toujours ✅
  • Créer la documentation TYPESCRIPT_GO.md
  • Tester lint et compilation ✅
  • Corriger la commande version dans le workflow ✅
  • Corriger le package-lock.json pour éviter les erreurs de build rollup ✅

Résumé final

✅ Packages ajoutés

✅ Scripts npm ajoutés

  • type-check:tsgo - Teste la compilation avec TypeScript Go (Version 7.0.0)

✅ Workflows CI

  • .github/workflows/tsgo-test.yml - Workflow dédié pour tester TypeScript Go
  • .github/workflows/ci.yml - Ajout de l'affichage de la version TypeScript

✅ Documentation

  • TYPESCRIPT_GO.md - Guide complet pour utiliser TypeScript Go dans le projet

🔧 Correctif

  • Mise à jour du package-lock.json pour utiliser les versions exactes des dépendances sans upgrader lockfileVersion
  • Le build rollup fonctionne maintenant sans erreurs sur les fichiers flow-tests

Commandes pour tester

# Vérifier la version de TypeScript Go
npx tsgo -v

# Tester la compilation avec TypeScript Go
npm run type-check:tsgo

# Build standard
npm run build

État actuel

✅ TypeScript Go v7.0.0-dev.20251027.1 fonctionne
✅ TypeScript standard v5.7.3 fonctionne normalement
✅ Les workflows CI sont configurés
✅ La documentation est complète
✅ Le build rollup fonctionne sans erreurs
✅ Le projet est prêt pour les tests TypeScript Go

Original prompt

Je veux tester si la version de Typescript 7 "ts-go" fonctionne.
Crée une branche dans laquelle tu modifie la version de Typescript et le workflow de test.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update TypeScript version to 7 'ts-go' and modify test workflow Add TypeScript Go (ts-go) testing support with @typescript/native-preview Oct 28, 2025
Copilot AI requested a review from jdeniau October 28, 2025 20:06
@jdeniau
Copy link
Member

jdeniau commented Oct 28, 2025

@copilot There is a lot of error while building with rollup:

/home/runner/work/immutable-js/immutable-js/src/Immutable.js → dist/immutable.js, dist/immutable.min.js, dist/immutable.es.js...
(!) [plugin typescript] type-definitions/flow-tests/covariance.js (6:6): @rollup/plugin-typescript TS8010: Type annotations can only be used in TypeScript files.
/home/runner/work/immutable-js/immutable-js/type-definitions/flow-tests/covariance.js:6:6

6   x: number;
       ~~~~~~

(!) [plugin typescript] type-definitions/flow-tests/covariance.js (9:6): @rollup/plugin-typescript TS8010: Type annotations can only be used in TypeScript files.
/home/runner/work/immutable-js/immutable-js/type-definitions/flow-tests/covariance.js:9:6

9   y: string;
       ~~~~~~

Copy link
Author

Copilot AI commented Oct 28, 2025 •
edited
Loading

@copilot There is a lot of error while building with rollup:

...

Fixed in d7e2e0e. The issue was caused by my initial npm install upgrading the package-lock.json lockfileVersion from 2 to 3, which also upgraded many dependencies and caused TypeScript to be stricter about Flow type annotations in .js files.

I've now corrected the package-lock.json to only add the @typescript/native-preview package without upgrading other dependencies. The rollup build now completes successfully without errors.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • bundlephobia.com
    • Triggering command: node ./resources/dist-stats.mjs (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jdeniau jdeniau Awaiting requested review from jdeniau

Assignees

@jdeniau jdeniau

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jdeniau