2026-04-30
- talk with Ant about yesterday’s account resources secret PR
- 6515
- update existing PR for time passing
- can I
- get both stateful and stateless stack working?
- inc. reg tests passing
- demonstrate small / no difference between sam and cdk
- create pr to relinquish management of stateful from sam
- create pr + manual script to adopt mgmt by cdk (probably manual script then pr merge)
- get both stateful and stateless stack working?
2026-04-29
- account resources PR to create SSv3 secret in CDK
- TODO: adopt from PfP replacing SAM dependency once rolled out
- review for complex typescript dependabot for Jim
2026-04-28
- most of the day chasing dependabots
- some chat with Ant on next stage for WIP
- 6055: wait for CDK, it will be easier
- 6257: cleared the middy / powertools incompatibilities at last
2026-04-27
- dependabot PRs stuck (PSU: 26, PfP: 22)
- why this created with wrong title?
https://github.com/NHSDigital/eps-prescription-status-update-api/pull/2994
- copilot just says ’temp ui issue’ but created twice that way 🤔
- updated manually and push empty commit to trigger rebuild
- ✅
- Upgrade: [dependabot] - bump @typescript-eslint/eslint-plugin from 8.58.1 to 8.58.2 #12111
- QC failure (suspect needs merging with other lint ecosystem) https://github.com/NHSDigital/eps-prescription-status-update-api/actions/runs/24953178291/job/73066823001?pr=3003
- additionally merged:
- “@typescript-eslint/parser”: “^8.58.2”,
- “typescript-eslint”: “^8.58.2”
- but not eslint 10.2.1 (yet)
- Strangely, the same change in PfP passes QC, but stuck on reg tests
https://github.com/NHSDigital/prescriptionsforpatients/pull/2589
- error signing prescription was previously an APIM thing, wait and see if resolves.
- why this created with wrong title?
https://github.com/NHSDigital/eps-prescription-status-update-api/pull/2994
2026-04-24
- 6394: notifications turn on
- release cut to int
- 2 hours! checking into Kayal’s tests
- 6257
- prep ci and release yml, review, prep pr
- PR to refactor SnsAlarm into cdk-utils
2026-04-23
Rubber duck
- use lambda invoke (like assist me) to solve gsul access from regression tests
PfP CDK
- merged!
- work on ci and release
PSU CDK
- updated the copilot instructions and got it to do same style as PfP
PSU release cut
- botched the Apotec PR
- second release stuck
PSU release cut is broken :cry: :https://github.com/NHSDigital/eps-prescription-status-update-api/actions/runs/24843652436/job/72728796355 Multiple failures on regression test step "When I sign the prescription": https://github.com/NHSDigital/electronic-prescription-service-api-regression-tests/actions/runs/24844899928/job/72729168579 Think I will need help to recover
2026-04-22
- meetings day
- show and tell
- notifications rollout
- pilot: 34
- extended with Asda sites making ~130
- last time: first entire install base of a supplier (Cegedim)
- now: Apotec (35) and Bootsi (819 or 1316):
- CDK
- modernise, standardise
- show range of resources
- regression tests
- review, Matt
- show pr flow
- notifications rollout
- release cut
- regression tests failed
- 6055
- merge failed on reg tests
2026-04-21
- cdk
- see if can simplify createMetricFilter in Alarms.ts
- yes did that
- address comments from Matt
- do not create artificial properties and consts to placate Sonar, ESlint
- root cause is CDK classes with side effects just by being created
- narrow everything to specifics (eg interface Props)
- state machine should read procedurally
- do not create artificial properties and consts to placate Sonar, ESlint
- see if can simplify createMetricFilter in Alarms.ts
- switch name to id in
prod- except that the code was not released
- which was just as well because would have enabled all 6 suppliers
2026-04-20
- cdk review comments on PfP
2026-04-17
- standups
- 6515
- script to compare cloudfoundation produced by sam and cdk
- remove differences so cdk is functionally equivalent
- boots rollout conversations, will need to be excludes not includes due to number
2026-04-16
- accommodate review comments on CDK
- push thru aws-lambda dep removal (old)
- dependabot review for Jim
- 6515: stateful psu cdk
- initial cut
- merge into single app (2 stacks) and DRY workflows
2026-04-15: Digmeds day
- up at 5, back at midnight
- feedback on EPS costing less than expected with more benefits (80 by 25-26 cf 75m expected by 29-30?)
- UCLH rolling out EPS issued scripts from ED from Aug.
- Total (all supplier’s trusts) will be 1m scripts going to communty pharmacy pa
- cf 1b currently, aka drop in the ocean
- Cost centre codes
- 5 digits in secondary care (except some community work), 6 in primary
- why not assign codes at individual consultant level and roll up as needed by diff audiences?
2026-04-14
- 6385: Still awaiting powertools build
- 6515: PSU CDK stateful
- fix: warnings about overriding state machine method responses https://github.com/NHSDigital/eps-cdk-utils/pull/667
- fix: sonar issues like new LambdaResource unassigned
2026-04-13
- rubber duck
- do we need to change the required gates from sam code?
- yes, work with Ant on adopt CDK ticket
- remove synth test as
quality_checks/IaC-validationdoes that
- standup
- Matt said happy to merge and fix: push for a merge tomorrow
- CDK
copilot --resume=72f4b433-0a89-4f6f-92ad-efe83e4d76fb- check proxygen running atop CDK
- lots of dependabot resolutions
- vitest 4 needs to be merged with coverage-v8
- typescript 6 requires bundler, removal of basedir, addition of types: [ “node” ]
2026-04-10
- rubber duck
- rollout PfP CDK:
- Matt said go for it once separate stack in place and merged
- rollout PfP CDK:
- some small review activity (ODS code rollout for Jim)
- CDK
- lots of sonar fixes after adding cdk package to scan
- PR out for review
- begin on CDK rollout, key finding is calculateVersionedStackName, already in use!
2026-04-09
- CDK
- if !process.env.DO NOT GET AWS EXPORT
- some resources created after synth (cross-region refs)
- local app v similar to ‘main’ (AWS) app
2026-04-08
- review and planning plus standup
- CDK
- regression tests passing in ci, but couple of failures locally
- merge latest workflows and rework
- remove tarball in favour of actual build (1.6.1)
2026-04-07
- rubber goose, standup
- update tickets with status for QA
- CDK for PfP
- fix common workflows
- fix corrupted tarball
- fix incompatible npm and lock file
- update sprint review doc for DM
- meeting about overnight downloads
2026-04-03 & 06: bank holiday
2026-04-02
- rubber goose, standup, release cut ~1.5h
- 4652: FHIR review for Bence ~1h (more)
- 6428: app name -> id review for Jim ~1h (less)
- CDK
- merge workflows from Jim
- rework to be conditional on PR title containing [CDK]
- obviously need to rework again before merge but at least get past reg tests
- fail workflow due to missing and then failing integrity tarball
- need to do cdk utils release: failing due to permissions since Jan
https://github.com/NHSDigital/eps-cdk-utils/actions/runs/23839167208/job/69490558606
- suggest PR on https://github.com/NHSDigital/eps-common-workflows/blob/main/.github/workflows/tag-release-devcontainer.yml
jobs: tag_release: permissions: id-token: write contents: write packages: write
- suggest PR on https://github.com/NHSDigital/eps-common-workflows/blob/main/.github/workflows/tag-release-devcontainer.yml
2026-04-01
time tracking
- https://nhsd-confluence.digital.nhs.uk/spaces/APIMC/pages/402855308/Time+Logging
- log travel and day against digimeds
CDK
- need to do alerts as well, Ant has example
- merge workflow changes from Jim