michael-s-molina opened a new pull request, #38472:
URL: https://github.com/apache/superset/pull/38472
### SUMMARY
Fixes and updates the extension developer documentation to match the actual
extension system implementation, and corrects the `superset-extensions-cli`
scaffold.
**Documentation**
Several docs contained stale or incorrect patterns copied from earlier
designs (e.g., VS Code-style `activate(context)` lifecycle, `extension.json`
frontend contribution blocks, wrong API method names and signatures). Updated
all extension docs to reflect how the system actually works today.
**`superset-extensions-cli` — remove `__init__.py` generation**
The CLI scaffold was generating `__init__.py` files inside the
`superset_extensions/` and `superset_extensions/{publisher}/` namespace
directories. This is incorrect: these directories are Python namespace packages
(PEP 420), and adding `__init__.py` to them actually breaks the namespace
package behavior. Without `__init__.py`, multiple independently installed
extensions can all contribute to the same `superset_extensions` namespace. With
it, only the first package on `sys.path` wins and the others are silently
ignored.
**Docs site infrastructure**
Fixed `@apache-superset/core/ui` references throughout the docs build
infrastructure — this subpath does not exist in the package. The correct paths
are `@apache-superset/core/components` for UI components and
`@apache-superset/core/theme` for `themeObject`.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A — documentation and tooling changes only.
### TESTING INSTRUCTIONS
1. Build the docs site and verify the extensions section renders correctly:
```bash
cd docs && yarn start
```
2. Run `superset-extensions init` and confirm the scaffolded project has no
`__init__.py` in `superset_extensions/` or `superset_extensions/{publisher}/`.
### ADDITIONAL INFORMATION
<!--- Check any relevant boxes with "x" -->
<!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in
[SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] Migration is atomic, supports rollback & is backwards-compatible
- [ ] Confirm DB migration upgrade and downgrade tested
- [ ] Runtime estimates and downtime expectations provided
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
