Improve JSModuleAnalyzer to support local names for dependency detection
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- build-system, tooling
Línea de trabajo
Start by locating the JSModuleAnalyzer in ui5-builder and reading how it currently recognizes known APIs and resolves dependency calls. Use the examples in the issue to identify the local-name cases that are missed. Done means dependencies are detected when APIs such as jQuery.sap.require or sap.ui.require are referenced through local variables.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
Currently, the UI5 Tooling is not able to detect all dependencies of a JavaScript module when it analyzes the code.
When using local names for certain APIs, such as jQuery.sap.require or sap.ui.require, the calls are not recognized and the dependencies are not detected.
Describe the solution you'd like
The underlaying JSModuleAnalyzer (part of ui5-builder) should be improved in a way that allows recognition of the known APIs in case they are using a local variable name.
Describe alternatives you've considered
Additional context
One example (there are many more use cases)
Dependency to my/module.js is detected
sap.ui.define(["jquery.sap.global"], function() {
jQuery.sap.require("my.module");
});
Dependency to my/module.js is NOT detected
sap.ui.define(["jquery.sap.global"], function(jq) {
jq.sap.require("my.module");
});
- Lenguaje dominante
- JavaScript
- Estrellas
- 511
- Forks
- 83
- Merge medio
- 1 d 5 h
- PR fusionados (30 d)
- 61
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de UI5/cli
-
Chromium 153+: UTF-8 BOM kept by `@ui5/builder` in bundles breaks appsPosiblemente ocupada @flovogt la tomó hace 17 días. Abiertomodule/ui5-builder
UI5/cli#1601 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Build cache: buildThemes cache is invalidated by irrelevant library.js / .library content changesAbiertomodule/ui5-builder module/ui5-fs module/ui5-project
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
module/ui5-project
Dificultad 4/5 3-5 días Aptitud para principiantes 64/100
Los mantenedores suelen responder en 1 día
-
[ROADMAP] UI5 CLI 6.0 / Specification Version 6.0Quizá libre de nuevo @flovogt la tomó hace 59 días y no hay ningún pull request abierto. Abiertoroadmap
UI5/cli#1485 · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
build excludes from application project are not applied to dependenciesQuizá libre de nuevo @flovogt la tomó hace 33 días y no hay ningún pull request abierto. Abierto
UI5/cli#1446 · 3 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Issues similares
-
automated issue report
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
lirantal/discoprint#36 ·
Los mantenedores suelen responder en 1 día
-
accepting PR Content:HTML
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
mdn/content#45988 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
pnpm/pnpm#16635 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
txn2/mcp-data-platform#2030 ·
Los mantenedores suelen responder en 1 día
-
ci hacktoberfest help wanted size/small type/bug type/docs type/enhancement up-for-grabs
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Redo-San/RedoSan-Authenticity#527 ·
Los mantenedores suelen responder en 1 día