Commit 98100d7
authored
fix(strands-memory): restore positional arg order in AgentCoreMemorySessionManager.__init__ (aws#318)
PR aws#288 inserted `converter` as the 2nd positional parameter, shifting
`region_name` from position 2 to 3. Any caller passing region_name
positionally (e.g. `AgentCoreMemorySessionManager(config, REGION)`)
silently assigned the region string to `converter`, leaving
region_name=None and causing botocore.exceptions.NoRegionError.
Move `converter` after the existing positional params and make it
keyword-only (after `*`) to restore backward compatibility and prevent
future positional misuse.1 parent 84d2916 commit 98100d7
1 file changed
Lines changed: 4 additions & 3 deletions
Lines changed: 4 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
104 | 104 | | |
105 | 105 | | |
106 | 106 | | |
107 | | - | |
108 | 107 | | |
109 | 108 | | |
110 | 109 | | |
| 110 | + | |
| 111 | + | |
111 | 112 | | |
112 | 113 | | |
113 | 114 | | |
114 | 115 | | |
115 | 116 | | |
116 | 117 | | |
117 | | - | |
118 | | - | |
119 | 118 | | |
120 | 119 | | |
121 | 120 | | |
122 | 121 | | |
| 122 | + | |
| 123 | + | |
123 | 124 | | |
124 | 125 | | |
125 | 126 | | |
| |||
0 commit comments