feat: improve skill scores and add automated review #1944
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| # Tessl Skill Review — runs on PRs that change any SKILL.md; posts scores as one PR comment. | ||
| # Docs: https://github.com/tesslio/skill-review | ||
| name: Tessl Skill Review | ||
|
|
||
| on: | ||
| pull_request: | ||
| branches: [development] | ||
| paths: | ||
| - "**/SKILL.md" | ||
|
|
||
| jobs: | ||
| review: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| pull-requests: write | ||
| contents: read | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: tesslio/skill-review@main | ||
| # Optional quality gate (off by default — do not enable unless user asked): | ||
| # with: | ||
| # fail-threshold: 70 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,21 +1,19 @@ | ||||||
| --- | ||||||
| name: antigravity | ||||||
| description: "Scaffold, build, test, and deploy Tari Ootle templates (Rust → WASM smart contracts) on Tari L2. Covers template authoring with #[template] macros, ResourceBuilder for fungible/NFT assets, Vault/Bucket lifecycle, transaction manifests, cross-component calls, access rules, client-side transaction building with ootle-rs, and local testing with tari_template_test_tooling. Use when developing, debugging, or deploying Tari Ootle templates and components." | ||||||
| --- | ||||||
|
|
||||||
| ## Overview | ||||||
|
|
||||||
| Tari Ootle is a decentralized application platform on Tari Layer 2. Build **templates** (smart contracts) in Rust, compile to WASM (`wasm32-unknown-unknown`), publish to the network, and interact with deployed **components** via transactions. | ||||||
|
|
||||||
| **Key concepts:** | ||||||
| - **Template** — Rust module annotated with `#[template]`; defines logic and state. Compiled to WASM and deployed. | ||||||
| - **Component** — Live instance of a template on-chain. Holds state and exposes public methods. | ||||||
| - **Resource** — Native digital asset (fungible token or NFT). Created with `ResourceBuilder`. | ||||||
|
Contributor
There was a problem hiding this comment. While simplifying the key concepts is good, removing the phrase 'Cannot be copied or accidentally destroyed' for
Suggested change
|
||||||
| - **Vault** — On-chain container for exactly one resource type. Must be stored in a component before the function returns. | ||||||
| - **Bucket** — Temporary container for resources during a transaction. Moves assets between vaults or method calls. | ||||||
| - **Transaction** — Set of instructions (CallFunction, CallMethod, etc.) signed, submitted, and executed atomically. | ||||||
|
|
||||||
| **Crate ecosystem:** | ||||||
|
|
||||||
|
|
@@ -303,8 +301,8 @@ pub fn do_something(&mut self, value: u64) { | |||||
| ## Resources | ||||||
|
|
||||||
| There are 4 resource types: | ||||||
| - **Public Fungible** — Interchangeable tokens, amounts visible on-chain | ||||||
| - **Public Non-Fungible** — Unique tokens, metadata visible on-chain | ||||||
| - **Confidential** — Fungible tokens with hidden amounts (Pedersen commitments) | ||||||
| - **Stealth** — Confidential UTXOs with hidden owners (TARI/tTARI is a stealth resource) | ||||||
|
|
||||||
|
|
||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,19 +1,19 @@ | ||||||
| --- | ||||||
| name: tari-ootle-cc | ||||||
| description: "Scaffold, build, test, and deploy Tari Ootle templates (Rust → WASM smart contracts) on Tari L2. Covers template authoring with #[template] macros, ResourceBuilder for fungible/NFT assets, Vault/Bucket lifecycle, transaction manifests, cross-component calls, access rules, client-side transaction building with ootle-rs, and local testing with tari_template_test_tooling. Use when developing, debugging, or deploying Tari Ootle templates and components." | ||||||
| --- | ||||||
|
|
||||||
| ## Overview | ||||||
|
|
||||||
| Tari Ootle is a decentralized application platform on Tari Layer 2. Build **templates** (smart contracts) in Rust, compile to WASM (`wasm32-unknown-unknown`), publish to the network, and interact with deployed **components** via transactions. | ||||||
|
|
||||||
| **Key concepts:** | ||||||
| - **Template** — Rust module annotated with `#[template]`; defines logic and state. Compiled to WASM and deployed. | ||||||
| - **Component** — Live instance of a template on-chain. Holds state and exposes public methods. | ||||||
| - **Resource** — Native digital asset (fungible token or NFT). Created with `ResourceBuilder`. | ||||||
|
Contributor
There was a problem hiding this comment. While simplifying the key concepts is good, removing the phrase 'Cannot be copied or accidentally destroyed' for
Suggested change
|
||||||
| - **Vault** — On-chain container for exactly one resource type. Must be stored in a component before the function returns. | ||||||
| - **Bucket** — Temporary container for resources during a transaction. Moves assets between vaults or method calls. | ||||||
| - **Transaction** — Set of instructions (CallFunction, CallMethod, etc.) signed, submitted, and executed atomically. | ||||||
|
|
||||||
| **Crate ecosystem:** | ||||||
|
|
||||||
|
|
@@ -301,8 +301,8 @@ pub fn do_something(&mut self, value: u64) { | |||||
| ## Resources | ||||||
|
|
||||||
| There are 4 resource types: | ||||||
| - **Public Fungible** — Interchangeable tokens, amounts visible on-chain | ||||||
| - **Public Non-Fungible** — Unique tokens, metadata visible on-chain | ||||||
| - **Confidential** — Fungible tokens with hidden amounts (Pedersen commitments) | ||||||
| - **Stealth** — Confidential UTXOs with hidden owners (TARI/tTARI is a stealth resource) | ||||||
|
|
||||||
|
|
||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,21 +1,19 @@ | ||||||
| --- | ||||||
| name: github-copilot | ||||||
| description: "Scaffold, build, test, and deploy Tari Ootle templates (Rust → WASM smart contracts) on Tari L2. Covers template authoring with #[template] macros, ResourceBuilder for fungible/NFT assets, Vault/Bucket lifecycle, transaction manifests, cross-component calls, access rules, client-side transaction building with ootle-rs, and local testing with tari_template_test_tooling. Use when developing, debugging, or deploying Tari Ootle templates and components." | ||||||
| --- | ||||||
|
|
||||||
| ## Overview | ||||||
|
|
||||||
| Tari Ootle is a decentralized application platform on Tari Layer 2. Build **templates** (smart contracts) in Rust, compile to WASM (`wasm32-unknown-unknown`), publish to the network, and interact with deployed **components** via transactions. | ||||||
|
|
||||||
| **Key concepts:** | ||||||
| - **Template** — Rust module annotated with `#[template]`; defines logic and state. Compiled to WASM and deployed. | ||||||
| - **Component** — Live instance of a template on-chain. Holds state and exposes public methods. | ||||||
| - **Resource** — Native digital asset (fungible token or NFT). Created with `ResourceBuilder`. | ||||||
|
Contributor
There was a problem hiding this comment. While simplifying the key concepts is good, removing the phrase 'Cannot be copied or accidentally destroyed' for
Suggested change
|
||||||
| - **Vault** — On-chain container for exactly one resource type. Must be stored in a component before the function returns. | ||||||
| - **Bucket** — Temporary container for resources during a transaction. Moves assets between vaults or method calls. | ||||||
| - **Transaction** — Set of instructions (CallFunction, CallMethod, etc.) signed, submitted, and executed atomically. | ||||||
|
|
||||||
| **Crate ecosystem:** | ||||||
|
|
||||||
|
|
@@ -303,8 +301,8 @@ pub fn do_something(&mut self, value: u64) { | |||||
| ## Resources | ||||||
|
|
||||||
| There are 4 resource types: | ||||||
| - **Public Fungible** — Interchangeable tokens, amounts visible on-chain | ||||||
| - **Public Non-Fungible** — Unique tokens, metadata visible on-chain | ||||||
| - **Confidential** — Fungible tokens with hidden amounts (Pedersen commitments) | ||||||
| - **Stealth** — Confidential UTXOs with hidden owners (TARI/tTARI is a stealth resource) | ||||||
|
|
||||||
|
|
||||||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While simplifying the key concepts is good, removing the phrase 'Cannot be copied or accidentally destroyed' for
Resourcemight omit a crucial piece of information. This is a fundamental principle of resource-oriented programming and is important for developers to understand the asset safety guarantees of the platform. I'd suggest re-adding it for clarity.