JavaScript track: exercises 06 and 07 appear to be in the wrong order
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- javascript, typescript
- Domaine
- documentation
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.
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.
- Langage dominant
- Python
- Étoiles
- 54
- Forks
- 36
- Métriques de merge des PR
- Aucune PR mergée en 30 j
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.
Autres issues de SAP-samples/codejam-code-based-agents
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
SAP-samples/codejam-code-based-agents#37 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
Toutes les issues de SAP-samples/codejam-code-based-agents
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100