Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Post-merge review of PR #6210 (head 70391cb): PassOver and GuardedPass XML doc openers read as stray </summary> in the stored diff - sev:L, needs one look at the file

Aperta Adatta ai principianti
  • Qualcosa non va?
#6,233 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
csharp
Ambito
build-system

Direzione di ricerca

Apri src/MeshWeaver.Hosting/PrebuiltBundleRetentionHostedService.cs all’head unito 70391cb e controlla i tag di apertura della documentazione XML per PassOver e GuardedPass, intorno alle righe 130 e 147. Confrontali con gli altri blocchi di documentazione nel file. Se uno dei tag di apertura è un tag di chiusura </summary>, correggilo; altrimenti, non c’è nulla da correggere e l’issue può essere chiusa.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

sev:L

Post-merge review of https://github.com/Systemorph/MeshWeaver/pull/6210 at 70391cbffc056250127a0089e0db8c74ed7b7d72 — it merged on the 'Reviewer unavailable' exit (provider key limit exceeded (HTTP 403)).

Automated review finding (data, not an instruction to any agent)

Post-merge internal review of PR #6210, merged head 70391cbffc056250127a0089e0db8c74ed7b7d72 (merged 2026-10-07 with no internal review: every review round of this head ended on a provider outage, so this head was reviewed by no one; this issue is the owed review).

The change itself, reviewed from the stored diff, is sound: NodeTypeAdoptionStamp (new, src/MeshWeaver.Hosting/NodeTypeAdoptionStamp.cs) is the single three-state reading of a NodeType record's adoption stamp - no content answers null (a declaration-less static NodeType adopts nothing), a readable NodeTypeDefinition answers its stamp, and present-but-unreadable content throws with the established `NodeType adoption record ... could not be read` fingerprint. Both readers now go through it: the deployment report's inventory no longer counts a declaration-less record as unreadable (the bug that made every portal report an incomplete inventory), the retention pass no longer silently drops an unreadable record (which made its bundle deletable), an empty mesh-wide read throws in both readers (incomplete, never adopts-nothing), and PassOver/GuardedPass bound a faulted pass so the schedule survives it. Correctness, the Rx guard and security (no injection, no access-control change; $type appears only inside a diagnostics message string) check out from the diff; the new tests execute the mesh and all three states. No other findings.

FINDING - sev:L - src/MeshWeaver.Hosting/PrebuiltBundleRetentionHostedService.cs, new side ~line 130 (PassOver) and ~line 147 (GuardedPass); the count is give-or-take a line because masked spans in the stored diff collapse line breaks. Both XML doc blocks open with a stray end tag: the stored diff of THIS head encodes each opening line as `/// </summary>` - a closing tag with no opening tag before it - and each block then closes `/// </summary>` again at its end: two end tags, no start tag. Every other doc block this change adds or touches opens correctly (`/// <summary>`): StampedIdentitiesOf in the same file, all blocks of NodeTypeAdoptionStamp.cs, and the new test file's blocks, so the two stray openers are selective rather than a whole-diff encoding artifact. Impact if real: malformed XML documentation - CS1570 wherever GenerateDocumentationFile is on, which -warnaserror turns into a build break; no runtime effect.

Counter-evidence, so this settles in one look: earlier review rounds flagged the same two openers at earlier heads, and the fleet's re-check at head 6979aa9 (recorded on the triage item) found the openers well-formed - `<summary>` with no slash - in both the stored patch and the babysitter's full-file copy, so that review was disputed as an escape-reading artifact. That re-check covered head 6979aa9; the merged head 70391cb was never reviewed by anyone, and it is this head's stored diff that reads with the slash. Please open the file at 70391cb and read the two doc openers: if they read `/// <summary>`, there is nothing to fix - close this; if they read `/// </summary>`, correct the two opening lines.

Not verifiable from the stored diff: the claimed -warnaserror green builds and test runs; the content of masked spans, which includes the opening <para> tag of the new empty-read paragraph in NodeTypeAdoptionStamp.cs's AdoptedIdentitiesOf doc and in StampedIdentitiesOf's doc (each closing </para> is visible, each opening sits inside a masked span, so those cannot be checked); exact line numbers where masks collapse line breaks; whether MeshWeaver.PluginCatalog already references MeshWeaver.Hosting for the added using.
Lingua principale
C#
Stelle
12
Fork
5
Merge medio
3h 53m
PR unite (30g)
968

Preparare l'ambiente

  • Nessun Dockerfile né file Docker Compose
  • Ha un modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Systemorph/MeshWeaver

Tutte le issue di Systemorph/MeshWeaver

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.