kamiazya/scopes

Enhancement: Add support for alternative key-value delimiters in aspect filtering

Ouverte

#111 ouverte le 31 août 2025

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Kotlin (3 forks)auto 404
good first issuehacktoberfest

Métriques du dépôt

Stars
 (2 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Summary

Currently the --aspect option only supports colon () as the delimiter for key-value pairs. Some users might expect to use equals sign (=) as well.

Proposed Enhancement

Support both : and = delimiters for aspect filtering:

  • --aspect priority:high (current)
  • --aspect priority=high (proposed addition)

Implementation Notes

  • Update the parsing logic in ListCommand.kt to split on both delimiters
  • Ensure backwards compatibility
  • Update documentation to reflect both options

Background

This suggestion came from AI code review feedback on PR #109, specifically noting that CLI tools sometimes accept multiple delimiter formats for user convenience.

Acceptance Criteria

  • Support both :" and =` delimiters
  • Maintain backwards compatibility
  • Update help text and documentation
  • Add tests for both delimiter types

Guide contributeur

Direction de recherche
Inspectez la logique d'analyse dans ListCommand.kt pour modifier la séparation clé valeur afin d'accepter à la fois les deux points et le signe égal, tout en garantissant la rétrocompatibilité.
Stack technique
kotlin
Domaine
cli
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Kotlin
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70