Skip to content

fix: resolve $ref/$inline/$merge on root components - #21

Closed
Macawls wants to merge 1 commit into
dolmen-go:masterfrom
Macawls:fix/components-with-directive
Closed

Macawls wants to merge 1 commit into
dolmen-go:masterfrom
Macawls:fix/components-with-directive

Conversation

@Macawls

@Macawls Macawls commented Oct 11, 2026

Copy link
Copy Markdown

Summary

  • Fix: root-level components holding a \$ref, \$inline or \$merge directive is now expanded instead of being silently skipped
  • Add hasDirective helper to detect directive keys on a map value
  • Add testdata case 47-inline-components covering \$inline on root components

Closes #9

Test plan

  • gofmt -l refs.go util.go
  • go vet ./...
  • go build ./...
  • go test ./...

When the root-level "components" key holds a $ref, $inline or
$merge directive, it is now expanded instead of being skipped.

Closes dolmen-go#9
@dolmen

dolmen commented Oct 14, 2026

Copy link
Copy Markdown
Member

I don't need you to run claude code: I can do it by myself.

@dolmen dolmen closed this Oct 14, 2026
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.

YAML objects named "components" don't resolve

2 participants