Improve way to verify is a skill card in spec

[richardhjtan]

Refer to https://linear.app/cardstack/issue/CS-8056/cover-boxel-spec-for-skill

This PR is to improve the identification of skill cards in the spec preview component

We would determine the skill card when:

• the code ref is module under https://cardstack.com/base/skill-card
• the code ref name is SkillCard
• also cover the class that skill card is a subclass for another card - finding under the chain of card type

[github-actions]

Host Test Results

    1 files  ±0      1 suites  ±0   24m 15s ⏱️ +3s
798 tests +5  796 ✔️ +5  2 💤 ±0  0 ❌ ±0 
803 runs  +5  801 ✔️ +5  2 💤 ±0  0 ❌ ±0 

Results for commit 693f9cc. ± Comparison against base commit 1e9a06f.

♻️ This comment has been updated with latest results.

[lukemelia]

This seems like it would not cover a skill that is a subclass of another skill.

[tintinthong]

The original code was clearer that it didn't do much.

A possible way to fix this is to introduce specType data in card-type resource. Another way is to just recurse the chain on read and make a stopping condition based on the type data that is already

I'd recommend the 2nd way unless that is visibly inefficient

The test should have a 1-level super and a 2-level super

[richardhjtan]

Thanks @lukemelia @tintinthong for the feedback. I switch the approach by finding the chain in cardType.typeCache that consists the boxel based skill card module

[lukemelia]

Is this typeCache guaranteed to be populated when this check is performed?

[richardhjtan]

Is this typeCache guaranteed to be populated when this check is performed?

To be assured, I introduce a function to find the class matches the desired codeRef at the first level or recursively.

693f9cc