SAP-samples / SAP-samples/codejam-code-based-agents
JavaScript track: exercises 06 and 07 appear to be in the wrong order
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 54
- Forks
- 36
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Summary
06-discover-connected-crimes.md references code that's only introduced in 07-solve-the-crime.md:
- Around line 238, it calls
this.orchestrationClient.chatCompletion(...), butorchestrationClientas a class field is only declared and constructed in exercise 07's Step 1 (07-solve-the-crime.md, "AddorchestrationClientas a class field and initialize it in the constructor"). - Around line 213, it instructs learners to add the new node "before
leadDetectiveNode" — butleadDetectiveNodeis also only created in exercise 07.
Effect
Following the exercises in their numbered order (06 then 07), a learner's investigationWorkflow.ts wouldn't type-check at exercise 06's step — this.orchestrationClient doesn't exist as a class member yet, and leadDetectiveNode doesn't exist to insert before.
Suggested fix
Either:
- Swap the exercise order (make current 07's class-field/constructor setup + Lead Detective node come before current 06's Intelligence Researcher node), or
- Move the
orchestrationClientclass-field/constructor setup into exercise 06 itself, and have exercise 07 only addleadDetectiveNode.
Either way, the sequencing needs to match what the code in each step actually depends on.
Environment
Reproduced on a personal SAP AI Core tenant, JavaScript/TypeScript track, exercises 01–07 followed in numbered order.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par 06-discover-connected-crimes.md et 07-solve-the-crime.md, en comparant les étapes référencées de investigationWorkflow.ts et leur ordre numéroté. Vérifiez que orchestrationClient et leadDetectiveNode sont introduits avant que l’exercice 06 ne les utilise, puis suivez les exercices 01–07 pour confirmer que le workflow JavaScript/TypeScript passe la vérification de types à chaque étape.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, typescript
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Active
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 78/100