Skip to content

feat(config): TS config hygiene — drop decorator flags, native-flat ESLint, slim jira-tracing tsconfig - #1692

Open
kalverra wants to merge 1 commit into
feat/dependency-modernizationfrom
DX-5569/config-hygiene
Open

kalverra wants to merge 1 commit into
feat/dependency-modernizationfrom
DX-5569/config-hygiene

Conversation

@kalverra

@kalverra kalverra commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Phase A of the TS standards modernization plan (follow-up to #1688). Refs DX-5569.

  • Remove experimentalDecorators/emitDecoratorMetadata from tsconfig.base.json (no decorators in source).
  • Rewrite root eslint.config.mjs as native flat config: eslint-config-prettier imported directly, tseslint.config() helper, @eslint/js recommended scoped to JS files, tseslint recommended for TS, jsonc-eslint-parser inline. No FlatCompat; @eslint/eslintrc devDep removed.
  • Fix code surfaced by stricter tseslint recommended severities: unused vars and explicit anys in go-mod-validator/go-conditional-tests tests.
  • Slim libs/jira-tracing/tsconfig.json (110-line tsc --init template) to extends-base + project specifics.

Validation: nx run-many lint/build/test 58/58 green; prettier check clean.

Stack created with GitHub Stacks CLI • Give Feedback 💬

…SLint, slim jira-tracing tsconfig

- Remove experimentalDecorators/emitDecoratorMetadata from tsconfig.base.json
  (no decorators in source).
- Rewrite root eslint.config.mjs as native flat config: eslint-config-prettier
  imported directly, tseslint.config() helper, @eslint/js recommended scoped
  to JS files, tseslint recommended for TS, jsonc-eslint-parser inline. No
  FlatCompat; @eslint/eslintrc devDep removed.
- Fix code surfaced by stricter tseslint recommended severities: unused vars
  (skip-position destructuring, unused imports) and explicit anys in
  go-mod-validator/go-conditional-tests tests (vi.mocked, typed fixtures).
- Slim libs/jira-tracing/tsconfig.json (110-line tsc --init template) to
  extends-base + project specifics.

Validation: nx run-many lint/build/test 58/58 green; prettier check clean.

Refs DX-5569
@kalverra
kalverra added this pull request to stack #1694 October 8, 2026 19:45
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

💥 No Changeset

Latest commit: 36429bc

Merging this PR will not cause any packages to be released. If these changes should not cause updates to packages in this repo, this is fine 🙂

If these changes should be included in a release's CHANGELOG, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

You can also use gocs to generate changeset files locally.

@kalverra
kalverra marked this pull request as ready for review October 8, 2026 20:16
@kalverra
kalverra requested a review from a team as a code owner October 8, 2026 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant