toolkit: convert WFN.Value to value receiver

[crozzy]

When using the go sql driver WFN will not be recognized as a driver.Valuer (only *WFN). As WFN is direct value in claircore's structs this makes it fiddly, the change allows both pointer and value to benefit from Value().

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.43%. Comparing base (befa444) to head (53712a5).
Report is 4 commits behind head on main.

❗ Current head 53712a5 differs from pull request most recent head 3542340. Consider uploading reports for the commit 3542340 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1278   +/-   ##
=======================================
  Coverage   56.42%   56.43%           
=======================================
  Files         233      233           
  Lines       15355    15355           
=======================================
+ Hits         8664     8665    +1     
+ Misses       5806     5803    -3     
- Partials      885      887    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hdonnay]

I think this was the case in the initial migration into the toolkit module and that broke things -- double-check the git log

[crozzy]

It looks like it was added here, and existed in the old package at the same time here, with the value receiver, I can't seem to find any specifics about why the method was changed.

If I'm thinking about this correctly this should never be an issue with pgx and issues will only surface when implementing a datastore that leverages the sql/driver lib.