Skip to content

feat: add folder entries, migrate vaults to public API, and fix entry filtering #32

Draft
Raphaël Larivière (Sylfwood) wants to merge 5 commits intomasterfrom
devops/DEVOPS-3818
Draft

feat: add folder entries, migrate vaults to public API, and fix entry filtering #32
Raphaël Larivière (Sylfwood) wants to merge 5 commits intomasterfrom
devops/DEVOPS-3818

Conversation

@Sylfwood
Copy link
Contributor

@Sylfwood Raphaël Larivière (Sylfwood) commented Jan 16, 2026

  • Add EntryFolderService with CRUD for all 16 folder
    subtypes
  • Migrate vaults to public API endpoints
  • Add GetEntries method with name/path filtering
  • Refactor credential tests with table-driven pattern
  • Bump version to 0.16.0

…tering

- Add ErrUnsupportedEntryType custom error for unsupported entry types
- Add IsUnsupportedEntryType helper function
- Add private getEntries function with filtering options
- Add GetEntries/GetEntriesWithContext on EntryCredentialService
- Add Test_GetEntries integration test
- Convert CRUD tests to table-driven with testify assertions
- Add logging (t.Logf) for better test visibility
- Add exact name match test in GetEntries
- Standardize mock data with Go conventions (testuser, testpass, test-*)
- Reduce code from 747 to 263 lines
Add compatibility table for DVLS versions:
- 0.16.0+ requires DVLS 2026.x
- 0.15.0 supports DVLS 2024.x, 2025.x
- Use string-based enums matching API contract
- Add dynamic vault creation helper for tests
- Skip legacy tests when TEST_VAULT_ID not provided
- Add EntryFolderService with full CRUD operations
- Support all 16 folder subtypes (Company, Credentials, Customer, etc.)
- Add EntryFolderData struct with domain/username fields
- Add tests for all subtypes and nested folder hierarchy
@Sylfwood Raphaël Larivière (Sylfwood) changed the title feat(entries): add GetEntries method with name/path filter feat: add folder entries, migrate vaults to public API, and fix entry filtering Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant