Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Opening one project retires another project's imported skills

Ouverte
  • Quelque chose ne va pas ?
#1,456 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
75/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
go
Domaine
cli, tooling

Piste de recherche

Start in internal/resident/skills.go at ReconcileImportedSkills, especially the scan around lines 655 and 739, then run the provided TestTwoProjectsKeepTheirSkills test in internal/resident. Confirm that skills from both projects remain active while deleting A's folder retires A's fact on an appropriate scan. Also check the skills manual and invalidates metadata for the requested documentation updates.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

area:session area:tools bug sev:serious

Found on santos/dev2 at 008363c98 (#1410). It reaches dev when #1410 merges.

What happened

With memory on, project A's imported skill is active after A opens, then is superseded when project B opens using the same home store. B's scan cannot see A's folder but treats its absence from that scan as deletion. A's attachment chip can remain while its next turn silently omits the skill.

Replication

Deterministic (no model). Save this test as internal/resident/two_project_skill_test.go and run go test ./internal/resident -run TestTwoProjectsKeepTheirSkills -count=1:

package resident
import (
 "path/filepath"
 "testing"
)
func TestTwoProjectsKeepTheirSkills(t *testing.T) {
 home, a, b := t.TempDir(), t.TempDir(), t.TempDir()
 alpha := filepath.Join(a, ".agents", "skills", "alpha")
 beta := filepath.Join(b, ".agents", "skills", "beta")
 writeImportedSkill(t, alpha, "alpha", "Drafts project A reports")
 writeImportedSkill(t, beta, "beta", "Drafts project B reports")
 graph := openStore(t)
 ReconcileImportedSkills(graph, a, home)
 ReconcileImportedSkills(graph, b, home)
 if _, ok := importedFactByArtifact(t, graph, alpha); !ok { t.Fatal("project A skill retired") }
}

Today it fails with project A skill retired.

Where

internal/resident/skills.go:655, ReconcileImportedSkills, scans one project; at :739 it supersedes every imported active fact missing from that scan.

The fix

Reconcile only facts owned by the scanned project and shared home. Retire another project's fact only after checking its own artifact has actually disappeared.

Acceptance

  • e2e: two codeaf chat windows sharing one home retain both projects' /skill entries after either launches.
  • Unit: the inlined test passes; deleting A's folder still retires A's fact on an appropriate scan.
  • Update the skills manual and state the former cross-project retirement in invalidates.
Langage dominant
Go
Étoiles
115
Forks
14
Merge moyen
9 h 15 min
PR mergées (30 j)
467

Préparer son environnement

Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de Agent-Field/CodeAF

Toutes les issues de Agent-Field/CodeAF

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.