Skip to content

Commit 69c6c23

Browse files
committed
fix(skill): update build verification steps and clarify example usage
1 parent 125edee commit 69c6c23

File tree

1 file changed

+14
-2
lines changed
  • .agents/skills/add-sensor-component-v1

1 file changed

+14
-2
lines changed

.agents/skills/add-sensor-component-v1/SKILL.md

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -515,8 +515,19 @@ You can usually grab the product image url details from the Adafruit product API
515515
---
516516

517517
## Step 6 — Build and Verify
518+
Check if windows powershell/cmd first by testing `echo $env:USERNAME %WINDIR%`
518519

520+
windows powershell users:
521+
```powershell
522+
. "$env:userprofile/.platformio/penv/Scripts/activate.ps1"
523+
pio run -e adafruit_feather_esp32s3
524+
```
525+
526+
bash/other users:
519527
```bash
528+
# often the env needs sourcing first
529+
. ~/.platformio/penv/bin/activate.sh
530+
520531
# PlatformIO build for a common target
521532
pio run -e adafruit_feather_esp32s3
522533
```
@@ -542,7 +553,7 @@ Ensure all doxygen style is consistent with existing drivers.
542553

543554
## Step 8 — Create Pull Requests
544555

545-
Two separate PRs are needed:
556+
Two separate PRs are needed (mention the model used for the PRs in the description):
546557

547558
### PR 1: Wippersnapper_Components
548559
- Branch from main, add the component folder with `definition.json` and `image`
@@ -568,7 +579,7 @@ Two separate PRs are needed:
568579

569580
---
570581

571-
## Worked Example: TMP119
582+
## Worked (abridged) Example: TMP119 - Do not follow this example step-by-step, it's only an abbreviated demonstration of the above process
572583

573584
TMP119: TI high-accuracy temperature sensor, TMP117 variant (chip ID 0x2117 vs 0x0117).
574585

@@ -583,6 +594,7 @@ TMP119: TI high-accuracy temperature sensor, TMP117 variant (chip ID 0x2117 vs 0
583594
- **Library:** `Adafruit_TMP117` (contains `Adafruit_TMP119` class)
584595
- **Product URL:** `https://www.adafruit.com/product/6482`
585596
- **Docs URL:** `https://learn.adafruit.com/adafruit-tmp119-high-precision-temperature-sensor`
597+
- **Markdown learn guide:** `https://learn.adafruit.com/adafruit-tmp119-high-precision-temperature-sensor.md?view=all` (mentions `## Arduino` section with driver repo link `https://github.com/adafruit/Adafruit_TMP117` and `## Example Code` using `https://github.com/adafruit/Adafruit_TMP117/blob/master/examples/TMP119_basic_test/TMP119_basic_test.ino`.)
586598
- **I2C addresses:** 0x48, 0x49, 0x4A, 0x4B (same as TMP117, datasheet Table 7-1)
587599
- **Measures:** Temperature only → subcomponents: `ambient-temp`, `ambient-temp-fahrenheit`
588600
- **Closest driver:** `WipperSnapper_I2C_Driver_TMP117.h`

0 commit comments

Comments
 (0)