Conversation
There was a problem hiding this comment.
Pull request overview
Upgrades the repository’s TypeScript toolchain from 5.9.3 to 6.0.2, adds TS6-specific tsconfig adjustments to keep builds clean, and aligns version references across architecture/EOL/release documentation.
Changes:
- Bump
typescriptdependency to6.0.2(and lockfile update). - Update
tsconfig.jsonfor TS6 (ignoreDeprecations: "6.0"and explicit Node typings). - Refresh TypeScript version references across flowcharts, architecture docs, EOL strategy, and release drafter template.
Reviewed changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Adds TS6 compatibility options (ignoreDeprecations, types: ["node"]). |
| package.json | Bumps typescript devDependency to 6.0.2. |
| package-lock.json | Locks TypeScript to 6.0.2 (resolved + integrity updated). |
| FLOWCHART.md | Updates MCP server layer diagram label to TypeScript 6.0. |
| End-of-Life-Strategy.md | Updates TypeScript lifecycle/version references (mindmap/table/gantt). |
| docs/api/media/End-of-Life-Strategy.md | Mirrors the EOL strategy updates in the docs copy. |
| docs/api/documents/ARCHITECTURE.html | Updates generated architecture HTML references to TS 6.x / 6.0.x. |
| ARCHITECTURE.md | Updates stack/version-matrix references from TS 5.x/5.9.x to 6.x/6.0.x. |
| .github/release-drafter.yml | Updates “Built With” TypeScript version to 6.0.x. |
FLOWCHART.md
Outdated
|
|
||
| subgraph "MCP Server Layer" | ||
| MCPServer[⚙️ EP MCP Server<br/>TypeScript 5.7<br/>Node.js 25] | ||
| MCPServer[⚙️ EP MCP Server<br/>TypeScript 6.0<br/>Node.js 25] |
There was a problem hiding this comment.
The flowchart node labels the EP MCP Server as “TypeScript 6.0”. Since this repo only consumes the server as an npm package (european-parliament-mcp-server), the server’s internal compiler version is hard to assert and can easily drift. Consider replacing this with the package version (e.g. european-parliament-mcp-server@1.1.20) or removing the TypeScript version from the diagram.
| MCPServer[⚙️ EP MCP Server<br/>TypeScript 6.0<br/>Node.js 25] | |
| MCPServer[⚙️ EP MCP Server<br/>european-parliament-mcp-server (npm)<br/>Node.js 25] |
|
@copilot apply changes based on the comments in this thread |
…in FLOWCHART.md Agent-Logs-Url: https://github.com/Hack23/euparliamentmonitor/sessions/300e7b72-b179-4e7e-be49-384b4ae413de Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Applied the reviewer's suggestion in |
TypeScript 6.0.2 upgrade with TS6 breaking change fixes and documentation alignment across release drafter, end-of-life strategy, architecture docs, and flowcharts.
TS6 breaking change fixes (
tsconfig.json)ignoreDeprecations: "6.0"—moduleResolution: "node"andbaseUrlare deprecated in TS6 (removed in TS7)"types": ["node"]— TS6 no longer auto-includes@types/nodewith node10 resolution, causingCannot find name 'fs'/'path'/'process'across all source files{ "compilerOptions": { "ignoreDeprecations": "6.0", // ...existing options... "types": ["node"] } }Documentation updates
.github/release-drafter.yml— Built With: 5.9.x → 6.0.xARCHITECTURE.md— Technology stack table and version matrixEnd-of-Life-Strategy.md+docs/api/media/copy — Lifecycle entries, Gantt timeline, succession planning; also fixed stale release cadence ("~6-month" → "~12-month major cycles")FLOWCHART.md— Replaced EP MCP Server TypeScript version label with npm package name (european-parliament-mcp-server (npm)) since this repo only consumes the server as a dependencydocs/api/documents/ARCHITECTURE.html— Generated HTML version refsReference
Migration pattern follows Hack23/game#539, adapted for this project's Node.js-only (non-bundler) tsconfig.