docs/reports/FRIIDUMP_NATIVE_REPORT_CANDIDATE5_CORRECTIONS.md

   1 # FriiDump native-report candidate5 corrections
   2
   3 ## Evidence basis
   4
   5 Candidate4 completed its Windows MSVC build, all 12 native-report fixtures,
   6 JSON Schema validation, semantic validation, and help/options smoke. A live
   7 stock-firmware `HL-DT-ST CDRW/DVD GCC4244 B101` run then retrieved a disc seed
   8 successfully and was cancelled after 561 sectors.
   9
  10 The atomic `friidump-test-result.v1` report was valid and correctly recorded:
  11
  12 - `partial_dump` / `partial`;
  13 - `failure_stage: user_cancelled`;
  14 - 561 completed sectors;
  15 - 1,148,928 completed bytes;
  16 - 1.169 seconds;
  17 - hashes and artifact identity for the exact partial output;
  18 - `Measurement scope: partial_progress`.
  19
  20 Two user-facing defects remained:
  21
  22 1. The default report used the log-derived filename but discarded the final
  23    log directory, so an absolute output/log path still produced the report in
  24    FriiDump's process working directory.
  25 2. The HLDS console summary divided the full expected disc size by the partial
  26    elapsed time, producing a meaningless throughput value and labeling the
  27    expected disc sector count as if it were completed output.
  28
  29 ## Candidate5 behavior
  30
  31 Without a path override, the report is now created beside the final FriiDump
  32 log. A basename-only log still places the report in the process working
  33 directory.
  34
  35 ```text
  36 D:\captures\test.iso.log
  37 D:\captures\test.iso.friidump.json
  38 ```
  39
  40 The existing override rules remain unchanged:
  41
  42 ```text
  43 --report-json <file>       Override the complete report pathname
  44 --report-dir <directory>   Override only the destination directory
  45 --firmware-modified <note> Mark firmware modified; omission assumes stock
  46 ```
  47
  48 For partial or cancelled HLDS runs, the console summary now separates:
  49
  50 - expected disc sectors;
  51 - completed output sectors;
  52 - completed output bytes;
  53 - cancellation sector versus a failed 16-sector read range;
  54 - throughput calculated only from completed bytes and the measured duration.
  55
  56 For the observed 561-sector result, the correct measured payload is
  57 1.095703125 MiB and the corresponding average at 1.169 seconds is about
  58 3374.28 MiB/h. Resumed runs continue to suppress throughput because their
  59 output byte count includes data from an earlier invocation.
  60
  61 ## Unchanged contract
  62
  63 The `friidump-test-result.v1` JSON contract is unchanged. The candidate4 live
  64 report already represented the partial output correctly; candidate5 corrects
  65 report placement and the human-readable console summary.