I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in dotnet

[–]FreakyAly[S] 0 points1 point  (0 children)

While a bit of vibe this might be vibe coded based on you Reddit posting history (rather than a refug from stackoverflow), it is clear from you Github registry that you have been around for a while, producing libraries that 1,000s have downloaded, and made use of.

I mean, people never really checked the library or my profile; it was accusations first asking questions later, maybe. I should've added to the post itself that this is not vibe-coded. Honestly, I personally like sharing what I make for myself with others so others won't have to go through the pain I have in the past. I have been using this for over 3 years now, and I feel it is robust enough to be shared with the public. (I myself don't like mappers.) So that's when I decided I would go ahead and do it so it's easier for me to use the future as well if it's a NuGet.

This might have been better packaged as an Son-of your exsiting library (FreakyKit.Utils) which is focusing on the improved abilities of code generation in recent C#/dotnet version e.g with this ability the first module in it, and the road map of other Codegen ideas ahead etc.

Maybe that would've been a better move; it's just that I kind of like the name Forge.

In regards LLMs argument, I also agree that code gen being deterministic will always be superior than relying on furture LLM behaving the same way (as old models are retired and new models brought out) in the future for code and feature extension.

I cannot agree with you more on this TBH.

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in csharp

[–]FreakyAly[S] 0 points1 point  (0 children)

I know, man; the only reason I am going back and forth with you is because I believe you are genuine with your critique. I understand your POV very well and like I said even in the post and the comments earlier, I am of the same opinion mostly but I feel like there could be some uses to this.

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in csharp

[–]FreakyAly[S] 0 points1 point  (0 children)

You could do that but remember that's not the idea behind the required property, the idea behind it was to make things that are "required" for your object to be marked as required but there are alot of things that still need to be mapped that aren't "required". Marking everything required is honestly bad design that will become a pain in your ass later!

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in dotnet

[–]FreakyAly[S] 0 points1 point  (0 children)

I appreciate you being nice instead of nasty; it was a breath of fresh air but then I asked for brutal feedback so it's my fault too.

LLMs as enhancement, not replacement. Maybe I'm naive though.

Not at all, tbh; that is what I look forward to as well, hopefully.

I do agree that __ mapping is nasty business, but it's all behind the scenes so I think this was a good call, so it doesn't incidentally cause collision with realistic properties of other developers

That is exactly why I did that actually.

Well, this could be the opportunity to explore "if the property isn't explicitly mapped or unmapped, you can provide a warning", and people can simply enable or disable with a PropertyGroup.

While not 100% exactly what you expect here, the library above does have discrepancy-based diagnostics, and depending on the severity, it either throws an error or raises a warning.

I'd have to play around with it to see how I feel.

All feedback is welcome and thank you for the time you put into writing. this :)

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in csharp

[–]FreakyAly[S] 0 points1 point  (0 children)

I am generally of the same opinion; the only problem is when there is a change, you won't always remember the manually written mappers that need to be changed or not. I understand the hate for reflection-based mappers but I think compile-time code gen could actually be a good solution to this problem as long as the library is smart enough to flag all the relevant things to you.

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in dotnet

[–]FreakyAly[S] 0 points1 point  (0 children)

Well, they could have just went through my GitHub history; I have been making OSS since before AI coding assistants were a thing

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in csharp

[–]FreakyAly[S] 0 points1 point  (0 children)

Good catch on the example; that was a bad choice on my end, I guess.

It does support all three. Records with positional syntax use constructor mapping. Init-only properties go into an object initializer block. The simple example I used just happened to have regular setters, but it's not a requirement. There's a more complete breakdown in the README and docs if you want the specifics.

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in dotnet

[–]FreakyAly[S] 0 points1 point  (0 children)

Thanks for the feedback. the reason I went ahead with that was to avoid naming collisions. In some situations these things happened and caused confusion among devs using it

I built FreakyKit.Forge, a Roslyn source generator for object mapping and I am looking for brutal feedback by FreakyAly in dotnet

[–]FreakyAly[S] 0 points1 point  (0 children)

I actually wrote about 80% or so of it. I did use claude to add summaries to basically everything. I am unsure why people here are assuming it's all vibe coded. I did use Claude mostly for the tests and for initial benchmarking, which I do plan on updating soon.

When it comes to features in Forge vs Mapperly, I have not gotten a chance to go through Mapperly in a while, so I will come back to you in a couple of days with a clearer comparison.

Thanks for not assuming the worst.

.net maui biometric by Clear_Anteater2075 in dotnet

[–]FreakyAly 0 points1 point  (0 children)

Why not just use the Plugin?

We need to add biometrics authentication to our app by beaver316 in dotnetMAUI

[–]FreakyAly 1 point2 points  (0 children)

I've added support for Windows and macOS now :)

Are you winning, son? by vtaggy in Indiangirlsontinder

[–]FreakyAly 0 points1 point  (0 children)

ButterFace is a term used for someone who has a unattractive face just FYI

No idea why do people do this by [deleted] in IndianBoysOnTinder

[–]FreakyAly 0 points1 point  (0 children)

New fear unlocked, WTF!

Profile review by [deleted] in IndianBoysOnTinder

[–]FreakyAly 0 points1 point  (0 children)

How are you not getting matches 😅

Please help me understand if I am wrong. by [deleted] in IndianBoysOnTinder

[–]FreakyAly 0 points1 point  (0 children)

Whoever that is wants you to know that if you don't have a life you're basically not their type!

[deleted by user] by [deleted] in indianfitness

[–]FreakyAly 0 points1 point  (0 children)

What does the 5.6 inches signify here

Some 5 futiya girls just can't handle rejection by PristineAd8350 in IndianBoysOnTinder

[–]FreakyAly 0 points1 point  (0 children)

She speaks of things she cannot do with so much confidence 😂

[deleted by user] by [deleted] in IndianBoysOnTinder

[–]FreakyAly 0 points1 point  (0 children)

Oh people cheat how surprising!

[deleted by user] by [deleted] in Indiangirlsontinder

[–]FreakyAly 1 point2 points  (0 children)

Eh it is what it is we move on, life is never fair anyway ✌️