Skip to content

feat: Docs dependencies should include Mixins #6760

Description

@janerikmai

Prerequisites

Framework Output Target

Multiple / Other

Output Target Version

0.0.0

Stencil Version

4.43.5

Reproduction URL

https://codesandbox.io/p/devbox/stencil-docs-dependencies-with-mixins-forked-73cjvp

Describe the Bug

Having a mixin FormElementMixin. This has a method renderValidation that will render the component my-validation (class MyValidation).

Now having class MyInput extends Mixin(FormElementMixin) using the renderValidation method from FormElementMixin.

Running build with type docs-json I would have expected that MyInput somehow lists MyValidation as indirect dependency. But the dependencies node is empty. I assume the static analyzer is missing some deeper checks or my understanding of dependencies here is wrong.

Steps to Reproduce

Go Sandbox,

look docs.json in root

Expected Behavior

I would expect the analyzer does recognize the method call from extended class that renders the other component in my current component, so I expect docs.json for this component has a dependencies node listing the other component for the current.

Additional Context

No response

Logs


Metadata

Metadata

Assignees

No one assigned

    Labels

    Feature: Want this? Upvote it!This PR or Issue may be a great consideration for a future idea.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions