#fix : update patch.go to call PatchValue correctly

[the-nurk]

issue

PatchValue accepts (patch, val, ...), but was receiving (val, patch).

this is predicated on orig being the data getting patched, and patch being the patch plan, (as if generated by diff).

test

exec hof flow patch.cue
cmp stdout stdout.golden

-- stdout.golden --
P.next: {
    b: "b"
    e: {
            a: "a"
            b: "b"
            c: "c"
    }
    f: "new"
}

-- patch.cue --
package patch

// original thing being patched
o: {
    a: "a"
    b: "b"
    e: {
        a: "a"
        b: "b"
        d: "d"
    }
}

// update structure
p: {
    "-": {
        a: "a"    // remove field 'a'
    }
    e: {
        "-": {
            d: "d"    // remove nested field 'd'
        }
        "+": {
            c: "c"    // add new nested field 'c'
        }
    }
    "+": {
        f: "new"     // add new field 'f'
    }
}

@flow()
P: {
  @task(st.Patch)
  orig: o
  patch: p
  next: _
} @print(next)

// Result after patch
n: {
    b: "b"
    e: {
        a: "a"
        b: "b"
        c: "c"
    }
    f: "new"
}
// validation
n: P.next

[verdverm]

What's with the exec refactor?

Not opposed, can you add a bit about what those changes are for or improve?

[the-nurk]

didn't like this style tbh. i'm more of a clear, linear, conditional function dispatch kinda guy. despite some of my code sometimes not looking like that.

[the-nurk]

i'm like 99% sure this use of ProcessState before checking if it was available was causing a segmentation fault for me. i'm kinda passed it so i forgot the reproduction. i was moving fast so i just re-wrote it with some ai help in all honesty. i've read a decent amount of the codebase and had to hand stitch this together since claude didn't exactly do well. it mainly just helped with what i would imagine a cookie cutter implementation for a nice threaded approach might look like with room for tracking multiple channels. i'm no goru but it works so far :)

[the-nurk]

i can put it on my list to reproduce, but it might be a couple hundred thousand minutes.

[the-nurk]

i show probably review the things i submit and provide more context. @verdverm

[the-nurk]

"clean, linear, function dispatch"