Skip to content

Add manifest tooling support for app.config.ts#287

Open
Krisztiaan wants to merge 1 commit intoexpo:mainfrom
Krisztiaan:feat/app-config-ts-manifest-support
Open

Add manifest tooling support for app.config.ts#287
Krisztiaan wants to merge 1 commit intoexpo:mainfrom
Krisztiaan:feat/app-config-ts-manifest-support

Conversation

@Krisztiaan
Copy link
Copy Markdown

Summary

  • add manifest provider support for object-literal app.config.js and app.config.ts files
  • keep JSON manifest behavior intact while extending links, diagnostics, and completions to dynamic config files
  • add dynamic-config fixtures and focused VS Code e2e coverage for asset and local plugin workflows

Details

  • expand manifest selectors to include app.config.js and app.config.ts
  • add a lightweight TypeScript AST helper to find Expo config object literals, asset references, and plugin definitions
  • teach manifest links, diagnostics, asset completions, and plugin completions to use those helpers for dynamic configs
  • update the extension setting descriptions to reflect support for object-literal JS/TS configs

Verification

  • npm run build
  • npm run lint
  • VSCODE_EXPO_TEST_PATTERN=dynamicManifest npm run test:vscode

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