Skip to content

feat: Experimental angle brackets!#38

Merged
vic merged 2 commits intomainfrom
angle-brackets
Nov 5, 2025
Merged

feat: Experimental angle brackets!#38
vic merged 2 commits intomainfrom
angle-brackets

Conversation

@vic
Copy link
Owner

@vic vic commented Nov 5, 2025

Allows writing <foo/bar/baz> instead of den.aspects.foo.provides.bar.provides.baz or even den.aspects.foo._.bar._.baz.

See the tested example usage

Copilot AI review requested due to automatic review settings November 5, 2025 06:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces an experimental angle-brackets feature that provides a convenient shorthand syntax for accessing nested aspects through the __findFile mechanism. Instead of using verbose den._.import-tree._.host syntax, users can now use <import-tree/host> to reference deeply nested aspects.

  • Implements angleBrackets function to resolve angle bracket notation to nested aspect paths
  • Updates example configuration to demonstrate the new angle bracket syntax
  • Adds documentation explaining the alias notation and experimental feature

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
modules/lib.nix Implements the angleBrackets function and exposes __findFile for angle bracket resolution
templates/default/modules/_example/import-non-dendritic.nix Updates examples to use new angle bracket syntax instead of verbose den._. notation
README.md Documents the provides alias and mentions the experimental angle-brackets feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vic vic force-pushed the angle-brackets branch 2 times, most recently from 0ab3b52 to e6b9293 Compare November 5, 2025 06:24
Copilot AI review requested due to automatic review settings November 5, 2025 06:24
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vic vic marked this pull request as ready for review November 5, 2025 06:34
Copilot AI review requested due to automatic review settings November 5, 2025 06:34
@vic vic merged commit f02b7b2 into main Nov 5, 2025
3 checks passed
@vic vic deleted the angle-brackets branch November 5, 2025 06:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant