Skip to content

feat: use agreed generic instructions wording - #38

Merged
CMonnin merged 1 commit into
mainfrom
generic-instructions
Jul 31, 2026
Merged

feat: use agreed generic instructions wording#38
CMonnin merged 1 commit into
mainfrom
generic-instructions

Conversation

@CMonnin

@CMonnin CMonnin commented Jul 31, 2026

Copy link
Copy Markdown
Member

This repo's share of DouglasNeuroInformatics/CPPQ#74. Applies the wording @joshunrau proposed and @AnneAlmey accepted — "Follow the instructions on the next page" — to the seven CPP_SERIES instruments here whose details.instructions was generic filler. Instructions can't be dropped entirely, since ODC requires them.

Instrument Was Edition
FAGERSTRÖM_NICOTINE_DEPENDENCE "Please respond to every question" 1 → 2
AUDIT_C "Please respond to every question" 1 → 2
TEN_ITEM_PERSONALITY_INVENTORY "Please respond to every question" 1 → 2
GAD_7 "Please complete all questions" 1 → 2
ADHD_ASRS_1.1 "This is a self-rated instrument, please answer all questions." 1 → 2
CUDIT_R "Please fill out answer that best describe your cannabis usage." 1 → 2
OLDER_AMERICANS_RESOURCES_AND_SERVICES "Now, I would like to ask you some questions about your family and friends. Please complete all questions." 1 → 2

For CUDIT_R the instructions live in clientDetails rather than details — that's the block changed.

Two deliberate judgement calls

The last two rows had text that was arguably more than filler, and applying the agreed wording uniformly was chosen over per-instrument exceptions:

  • OLDER_AMERICANS_RESOURCES_AND_SERVICES — the "Now, I would like to ask you some questions about your family and friends" lead-in is meaningful; only the trailing "Please complete all questions" was filler.
  • CUDIT_R — meaningful, though ungrammatical as written.

Both are easy to restore or reword if @AnneAlmey would rather keep them — say so and I'll adjust.

Out of scope

  • All TMB_* interactive instruments — "Instructions will be presented on screen in the task." is accurate. (TMB_MATRIX_REASONING and TMB_EMOTION_RECOGNITION have no instructions field at all.)
  • ATHENS_INSOMNIA_SCALE — no instructions field.
  • C3Q, SIDAS, MODIFIED_SIDAS — not CPP_SERIES members.
  • PHQ_9, STARKSTEIN_APATHY_SCALE — instructions describe a recall period, not filler.

Series pins

These edition bumps do not reach participants on their own — CPP_SERIES pins exact editions. A follow-up PR in CPPQ updates the pins once this and its sibling PRs merge. See CPPQ#142 for the evidence that the pin is exact rather than a minimum.

Verification

pnpm exec tsc && pnpm exec eslint lib passes. The one unrelated hunk is a trailing blank line removed from AUDIT_C/index.ts by the repo's own prettier pre-commit hook.

🤖 Generated with Claude Code

Replaces filler instructions with the wording agreed by @joshunrau and
@AnneAlmey on DouglasNeuroInformatics/CPPQ#74: "Follow the instructions
on the next page". Instructions cannot be removed entirely as ODC
requires them.

Editions bumped so the change reaches participants.

Refs DouglasNeuroInformatics/CPPQ#74

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@CMonnin

CMonnin commented Jul 31, 2026

Copy link
Copy Markdown
Member Author

⚠️ Collides with #18 — both PRs edit lib/forms/FAGERSTRÖM_NICOTINE_DEPENDENCE/index.ts and both bump internal.edition from 1 to 2.

This will bump to edition 2. #18 will need to bump to edition 3

@CMonnin
CMonnin merged commit 1f22455 into main Jul 31, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant