avenj / bot-cobalt Goto Github PK

The existing Common.pm magic Exporter incantations are a bummer.

Most of the plain exporters are now Exporter::Tiny; Common.pm needs a custom import method to call Import::Into.

The existing manually-specified list of exported types in Common.pm should go away; we can just pull Types::Standard and List::Objects::Types ... probably Types::Path::Tiny also.

Existing libs using MooX types should remain compat, and attrs can be audited for custom isa subs that need to be updated to use Type::Tiny types, paths that should be coerced to Path::Tiny paths, ...

We may want our own type lib for Bot::Cobalt types.

Currently the core english.yml langset is appended to Lang.pm at build-time to provide defaults.

Other than being dirty, part of the move to Dist::Zilla involves dropping the extra unnecessary build-time steps.

Completed steps:

• Move etc/ to share/ and drop empty var/db/
• Drop scattered Lang.pm components (.pod, .pm.in, .pm.PL)
• Lang.pm attr cleanup

Pending steps:

• Lang.pm should use File::ShareDir and Path::Tiny
• cobalt2-installer rewrite
• Related t/ audit

Proc::PID::File is an unnecessary dependency; the cobalt2 frontend can handle PID file mgmt on its own.

Outgoing messages are currently rate-limited by POE::Component::IRC and there is no convenient way to change the behavior.

We at least need a configuration option to enable pocoirc's 'flood => 1' ... also perhaps our own optional outgoing queue managed via Object::RateLimiter.

SJM requested this a while ago, and I forgot :-(