Skip to content

Commit 8e74200

Browse files
committed
docs(skills): upgrade example stops claiming a connector sync ever ran
The worked R1 in the upgrade skill's examples walks a protocol-16 `connectors[].fieldMappings[].transform` source. Since the connector sync keys retired, the chain removes all of `fieldMappings` after the `transform` key, and the tombstone the author meets is that key's. The example said the connector had been landing raw values and its report template verified the decision with a "connector sync run"; no engine ever ran a connector-attached sync, so both now say what is true. The file shrinks by 37 bytes (1197 to 1188 tokens) at the same 119 lines. Claude-Session: https://claude.ai/code/session_01Sfe5YjBLwB9J3y8fvm2xq1 Co-authored-by: Claude <noreply@anthropic.com>
1 parent 2c41e48 commit 8e74200

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

‎skills/objectstack-upgrade/references/examples-upgrade.md‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ The shape in a protocol-16 project:
1616
}
1717
```
1818

19-
The chain deletes the key (`field-mapping-transform-removed`) and the schema
20-
tombstones it, so the parse error *is* the prescription: the union had five
21-
members and **no runtime ever executed any of them**. The customer wrote it
22-
because they wanted a transformation, and that need is real regardless.
19+
The chain deletes the key (`field-mapping-transform-removed`), then all of
20+
`fieldMappings` (`connector-sync-keys-removed`), whose tombstone *is* the
21+
prescription: **nothing ever ran any connector field mapping**. The customer
22+
wrote it because they wanted a transformation, and that need is real regardless.
2323

2424
The prescription names one live target; the rest is the business decision:
2525

@@ -29,10 +29,10 @@ The prescription names one live target; the rest is the business decision:
2929
| multi-source, multi-stage transformation | **nothing** — the L2 ETL layer retired at 17, unexecuted. Do it where it runs: warehouse ELT, a `flow`, a job. |
3030
| nothing — the value was already correct | delete the key and record that the transformation never ran. |
3131

32-
That third row is frequently the truth: the member never executed, so the
33-
connector has been landing raw values for as long as it has been running.
34-
Whether the downstream data is wrong is a question only the owner can answer —
35-
exactly the kind of finding the report exists to surface.
32+
That third row is frequently the truth: no connector sync ever ran, so no value
33+
ever passed through this mapping. Where the data really comes from, and whether
34+
it is right, only the owner can answer — exactly the kind of finding the report
35+
exists to surface.
3636

3737
### 3.4 The report — the human half
3838

@@ -64,7 +64,7 @@ _N sites, M conversions. Ported into sources from `os migrate meta --out`._
6464
- **Options:** import-mapping `transform` · ETL step · delete
6565
- **Decision:** delete — owner confirmed the values arrive pre-scaled.
6666
_Decided by: <who>, <date>._
67-
- **Verified:** `os validate` green; connector sync run against staging, 200 rows, values unchanged.
67+
- **Verified:** `os validate` green; nothing ever read the key, so no data changes.
6868

6969
## 3 · Open decisions
7070

0 commit comments

Comments
 (0)