Skip to content

refactor: use native sign instead dirctl - #148

Merged
arpad-csepi merged 6 commits into
mainfrom
refactor/replace-dirctl-with-cosign
Oct 21, 2026
Merged

arpad-csepi merged 6 commits into
mainfrom
refactor/replace-dirctl-with-cosign

Conversation

@arpad-csepi

@arpad-csepi arpad-csepi commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

These changes are to remove dirctl dependency as a binary and re-implement cosign related functionality directly in the SDK. New tests are added to ensure the sign and verify yields the same result in dirctl and the native implementation.

@arpad-csepi arpad-csepi self-assigned this Oct 20, 2026
Comment thread dir-sdk-python/agntcy/dir_sdk/tests/test_signing.py Dismissed
Comment thread dir-sdk-python/agntcy/dir_sdk/tests/test_signing.py Fixed
@codecov-commenter

codecov-commenter commented Oct 20, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Signed-off-by: Árpád Csepi <csepi.arpad@outlook.com>
@arpad-csepi
arpad-csepi force-pushed the refactor/replace-dirctl-with-cosign branch from d6ea741 to d7410f9 Compare October 20, 2026 11:38
Signed-off-by: Árpád Csepi <csepi.arpad@outlook.com>
@arpad-csepi
arpad-csepi force-pushed the refactor/replace-dirctl-with-cosign branch from 2e5bcf0 to 28244fd Compare October 20, 2026 11:43
Signed-off-by: Árpád Csepi <csepi.arpad@outlook.com>
Signed-off-by: Árpád Csepi <csepi.arpad@outlook.com>
Comment thread dir-sdk-python/agntcy/dir_sdk/tests/test_signing.py Fixed
Signed-off-by: Árpád Csepi <csepi.arpad@outlook.com>
@arpad-csepi
arpad-csepi force-pushed the refactor/replace-dirctl-with-cosign branch from 4c0c954 to dea7738 Compare October 20, 2026 12:11
Signed-off-by: Árpád Csepi <csepi.arpad@outlook.com>
@arpad-csepi arpad-csepi changed the title refactor: use cosign instead dirctl refactor: use native instead dirctl Oct 20, 2026
@arpad-csepi arpad-csepi changed the title refactor: use native instead dirctl refactor: use native sign instead dirctl Oct 20, 2026
@arpad-csepi arpad-csepi linked an issue Oct 20, 2026 that may be closed by this pull request
@arpad-csepi arpad-csepi added the enhancement New feature or request label Oct 20, 2026
@arpad-csepi arpad-csepi added this to SDK Oct 20, 2026
@arpad-csepi arpad-csepi removed this from SDK Oct 20, 2026
@arpad-csepi
arpad-csepi marked this pull request as ready for review October 20, 2026 16:57
@arpad-csepi
arpad-csepi requested a review from a team as a code owner October 20, 2026 16:57
@arpad-csepi
arpad-csepi merged commit 90b722a into main Oct 21, 2026
9 checks passed
@arpad-csepi
arpad-csepi deleted the refactor/replace-dirctl-with-cosign branch October 21, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove dirctl dependency

4 participants