ChewKeanHo / AutomataCI

An open-source, redistributable, template-guided, and semi-autonomous CI infrastructure readily available your next project.
Apache License 2.0
21 stars 1 forks source link

Purge All AI's Implementations #348

Open hollowaykeanho opened 4 months ago

hollowaykeanho commented 4 months ago

Description

That's it. It's done. No more AI implementations. This ticket tracks all AI implementations purging from this repository.

Expected Behavior

All AI implementations are purged of the source codes. AI implementations for source codes shall be banned. Documentations is still usable.

Current Behavior

The ill-implementations (overconfident AI) is still within the source codes causing major CI tasks failed.

Steps to Reproduce [COMPULSORY]

Run through all the source codes.

Associated Data Files

No response

corygalyna commented 3 months ago

? What's the story behind?

lucirox commented 3 months ago

Interesting.

hollowaykeanho commented 3 months ago

2 AIs are recommending broken PowerShell codes confidently which broke the Package API and some critical libraries. I was debugging until fall sick and finally realized the project is so screwed.

Currently purging all of the traces.

hollowaykeanho commented 3 months ago

Here's a case - AI confidently recommend the following:

on PowerShell:

if ([string]::IsNullOrEmpty($___target)) {
    ...
}

on POSIX Shell

if [ -z "$___target" ]; then
    ...
fi

with some non-sense reliability reasons so hestiaSTRING_Is_Empty is invented to unify the function for all shells.

However, after reading through all the specs, they can be done easily across all the shells:

On PowerShell

if ($___target -eq "") {
    ...
}

On POSIX Shell

if [ "$___target" = "" ]; then
    ...
fi

Affects: Entire project

hollowaykeanho commented 3 months ago

Another case:

AI confidently recommend a list directory loop using multiple cmdlet in PowerShell:

Get-ChildItem -Path $path -File -Recurse
    | Where-Object { $_.Name -like ".pdf" }
    | ForEach-Object {
     ...
}

for POSIX Shell:

for ___file in "$path/"*; do
    if [ ! -f "$___file" ]; then
        continue
    fi

    ...
done

The problem from what the AI recommended is continue cannot be used in the loop due to multiple cmdlet. Moreover, there is no need to use so many cmdlet for this case. After reading the specs, this is the correct PowerShell:

foreach ($___file in (Get-ChildItem -Path $path -File -Recurse -Filter ".pdf")) {
     ...
}

Affects: Entire project - deeply damaged DEB library as well.

corygalyna commented 3 months ago

That's the reason for re-writing all the libraries into Hestia? Yeah, I do agree with you. The AI screwed this up really bad.