-
Notifications
You must be signed in to change notification settings - Fork 18
feat: Add DelegateWithActions #146
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
de5da2b
feat: Add DelegateWithActions
snawaz 8edbf20
add new action format with selective encryption
snawaz 2346f9c
switch to libsodium-rs
snawaz b086b4e
split program extracting out dlp-sdk
snawaz 237eddc
Add MaybeEncryptedAccountMeta
snawaz b14a7f1
Add trait Encrypt
snawaz d563894
add encrypt.rs
snawaz 8e10682
replace create_post_delegation_actions with impl Encrypt
snawaz 814a975
add dev_experience
snawaz 23f1144
rename dlp-api to magicblock-delegation-program-api
snawaz f87cbee
switch to borsh
snawaz 86400e0
remove uncessary derive
snawaz 380a501
feat: impl from trait
Dodecahedr0x d03478c
feat: remove pubkey dep
Dodecahedr0x 93553ba
fix: disable default serde features
Dodecahedr0x d588b07
Add trait ClearText to be used onchain
snawaz b4979e2
Add trait ClearTextWithInsertable to be used onchain
snawaz 9625aba
cleanup
snawaz 6a52d90
fmt
snawaz e923cee
add .coderabbit.yaml
snawaz 6817eea
design features properly
snawaz e0022fa
address rabbit feedback
snawaz a53b394
use features in dlp-api as well
snawaz 97432c4
address rabbit feedback
snawaz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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,140 @@ | ||
| language: en-US | ||
| tone_instructions: '' | ||
| early_access: true | ||
| enable_free_tier: true | ||
| reviews: | ||
| # Review style + summary placement | ||
| profile: assertive | ||
| high_level_summary_placeholder: "<!-- coderabbit-summary -->" | ||
| review_status: false | ||
| collapse_walkthrough: true | ||
|
|
||
| # Noise reduction (these default to true; we explicitly turn them off) | ||
| changed_files_summary: false | ||
| sequence_diagrams: false | ||
| estimate_code_review_effort: false | ||
| related_issues: false | ||
| related_prs: false | ||
| suggested_labels: false | ||
| labeling_instructions: [] | ||
| auto_apply_labels: false | ||
| suggested_reviewers: true | ||
| auto_assign_reviewers: false | ||
| in_progress_fortune: false | ||
| poem: false | ||
| enable_prompt_for_ai_agents: true | ||
|
|
||
| # Auto-review behavior | ||
| auto_review: | ||
| enabled: true | ||
| drafts: true | ||
| auto_incremental_review: true | ||
| base_branches: [".*"] | ||
|
|
||
| # Don’t auto-generate docstrings/tests (reduce chatter) | ||
| finishing_touches: | ||
| docstrings: | ||
| enabled: false | ||
| unit_tests: | ||
| enabled: true | ||
| pre_merge_checks: | ||
| docstrings: | ||
| mode: off | ||
| title: | ||
| mode: off | ||
| description: | ||
| mode: off | ||
| issue_assessment: | ||
| mode: off | ||
|
|
||
| # Lightweight, high-signal tools for this repo | ||
| tools: | ||
| # Audited languages: Rust, Shell, TypeScript/JavaScript, Protobuf, Markdown, YAML, TOML, Makefile. | ||
| ast-grep: | ||
| rule_dirs: [] | ||
| util_dirs: [] | ||
| essential_rules: true | ||
| packages: [] | ||
| shellcheck: | ||
| enabled: true | ||
| markdownlint: | ||
| enabled: true | ||
| github-checks: | ||
| enabled: false | ||
| timeout_ms: 90000 | ||
| languagetool: | ||
| enabled: true | ||
| enabled_rules: [] | ||
| disabled_rules: [] | ||
| enabled_categories: [] | ||
| disabled_categories: [] | ||
| enabled_only: false | ||
| level: default | ||
| biome: | ||
| enabled: true | ||
| yamllint: | ||
| enabled: true | ||
| gitleaks: | ||
| enabled: true | ||
| eslint: | ||
| enabled: true | ||
| buf: | ||
| enabled: true | ||
| actionlint: | ||
| enabled: true | ||
| semgrep: | ||
| enabled: true | ||
| clippy: | ||
| enabled: true | ||
| oxc: | ||
| enabled: true | ||
| dotenvLint: | ||
| enabled: true | ||
| checkmake: | ||
| enabled: true | ||
| osvScanner: | ||
| enabled: true | ||
| path_instructions: | ||
| - path: "{dlp-api,src}/**" | ||
| instructions: | | ||
| Treat any usage of `.unwrap()` or `.expect()` in production Rust code as a MAJOR issue. | ||
| These should not be categorized as trivial or nit-level concerns. | ||
| Request proper error handling or explicit justification with invariants. | ||
| - path: "{tests}/**" | ||
| instructions: Usage of `.unwrap()` or `.expect()` in test code is acceptable and may be treated as trivial. | ||
| chat: | ||
| # We leave emoji/ASCII art, but no auto-replies | ||
| art: true | ||
| auto_reply: false | ||
|
|
||
| knowledge_base: | ||
| # Avoid web-search commentary in reviews | ||
| web_search: | ||
| enabled: false | ||
| code_guidelines: | ||
| enabled: true | ||
| filePatterns: [] | ||
| learnings: | ||
| scope: auto | ||
| issues: | ||
| scope: auto | ||
| jira: | ||
| usage: auto | ||
| project_keys: [] | ||
| linear: | ||
| usage: auto | ||
| team_keys: [] | ||
| pull_requests: | ||
| scope: auto | ||
| mcp: | ||
| usage: auto | ||
| disabled_servers: [] | ||
| code_generation: | ||
| docstrings: | ||
| language: en-US | ||
| path_instructions: [] | ||
| unit_tests: | ||
| path_instructions: [] | ||
| issue_enrichment: | ||
| planning: | ||
| enabled: true | ||
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
Oops, something went wrong.
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.
This note and setting disagree on whether unit-test generation is on.
Line 34 says this section is reducing chatter by not generating tests, but Line 39 keeps
unit_tests.enabled: true. Either flip the setting or rewrite the comment so the file is not self-contradictory.🤖 Prompt for AI Agents