Repository navigation
Commit 7f0fdf8
committed
fix(manifest): store full 40-char SHAs (PR #81 static review)
Static review on PR #81 caught a real correctness bug in the
manifest pinning: I stored 12-char abbreviated SHAs in
``external/MANIFEST.toml`` (e.g. ``dd5feaaaa500``), but
``scripts/verify_external_data.py:118-126`` compares
``spec["sha"]`` to ``git rev-parse HEAD`` with exact string
equality. ``git rev-parse HEAD`` returns the full 40-char object
ID, so every populated git-backed entry would FAIL verification
even when checked out to the intended commit. The
``Generate verifier report`` step in ``.github/workflows/
validation.yml`` would have failed on every workflow run, which
defeats the entire 1.8.1 pinning effort.
Fix
---
- Replace each 12-char abbreviation in ``external/MANIFEST.toml``
with the full 40-char SHA (read again from the same local
clones, this time without ``rev-parse --short``).
- Update the ``git checkout`` commands in ``.github/workflows/
validation.yml`` to use the full SHAs so the workflow and the
manifest are byte-identical references.
Verified locally
----------------
``python scripts/verify_external_data.py --strict`` now reports
``9 PASS, 0 WARN, 0 SKIP, 0 FAIL`` (previously was
``1 PASS, 8 WARN, 0 SKIP, 0 FAIL`` in 1.8.0 with TBD pins; the
abbreviated-SHA version of 1.8.1 would have flipped that to
``8 FAIL`` — exactly the contradiction the static-review pass
flagged).1 parent 4400299 commit 7f0fdf8
2 files changed
Lines changed: 21 additions & 16 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
87 | | - | |
88 | | - | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
89 | 92 | | |
90 | 93 | | |
91 | 94 | | |
92 | | - | |
| 95 | + | |
| 96 | + | |
93 | 97 | | |
94 | 98 | | |
95 | | - | |
96 | 99 | | |
97 | 100 | | |
98 | 101 | | |
99 | | - | |
| 102 | + | |
| 103 | + | |
100 | 104 | | |
101 | 105 | | |
102 | | - | |
103 | | - | |
| 106 | + | |
| 107 | + | |
104 | 108 | | |
105 | 109 | | |
106 | 110 | | |
107 | | - | |
| 111 | + | |
| 112 | + | |
108 | 113 | | |
109 | 114 | | |
110 | 115 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
| 43 | + | |
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
65 | | - | |
| 65 | + | |
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
72 | | - | |
| 72 | + | |
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
77 | 77 | | |
78 | 78 | | |
79 | 79 | | |
80 | | - | |
| 80 | + | |
81 | 81 | | |
82 | 82 | | |
83 | 83 | | |
| |||
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
90 | | - | |
| 90 | + | |
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
| |||
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
114 | | - | |
| 114 | + | |
115 | 115 | | |
116 | 116 | | |
117 | 117 | | |
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
121 | 121 | | |
122 | | - | |
| 122 | + | |
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
| |||
0 commit comments